Refactoring

Author: wcoder

src/highlightjs-line-numbers.js

@@ -1,3 +1,5 @@
+// jshint multistr:true
+
 (function (w, d) {
 	'use strict';
 
@@ -9,14 +11,6 @@
 	    DATA_ATTR_NAME = 'data-line-number',
 	    BREAK_LINE_REGEXP = /\r\n|\r|\n/g;
 
-	// string format
-	// https://wcoder.github.io/notes/string-format-for-string-formating-in-javascript
-	var format = function (str, args) {
-		return str.replace(/\{(\d+)\}/g, function(m, n){
-			return args[n] ? args[n] : m;
-		});
-	};
-
 	if (w.hljs) {
 		w.hljs.initLineNumbersOnLoad = initLineNumbersOnLoad;
 		w.hljs.lineNumbersBlock = lineNumbersBlock;
@@ -121,7 +115,7 @@
 	 * Doing deep passage on child nodes.
 	 * @param {HTMLElement} element
 	 */
-	function dublicateMiltilineNodes(element) {
+	function dublicateMiltilineNodes (element) {
 		var nodes = element.childNodes;
 		for (var node in nodes){
 			if (nodes.hasOwnProperty(node)) {
@@ -141,29 +135,41 @@
 	 * Method for fix multi-line elements implementation in highlight.js
 	 * @param {HTMLElement} element
 	 */
-	function dublicateMiltilineNode(element) {
+	function dublicateMiltilineNode (element) {
 		var className = element.parentNode.className;
 
 		if ( ! /hljs-/.test(className)) return;
 
 		var lines = getLines(element.textContent);
+
 		for (var i = 0, result = ''; i < lines.length; i++) {
 			result += format('<span class="{0}">{1}</span>\n', [ className, lines[i] ]);
 		}
 		element.parentNode.innerHTML = result.trim();
 	}
 
-	function getLines(text) {
+	function getLines (text) {
 		if (text.length === 0) return [];
 		return text.split(BREAK_LINE_REGEXP);
 	}
 
-	function getLinesCount(text) {
+	function getLinesCount (text) {
 		return (text.trim().match(BREAK_LINE_REGEXP) || []).length;
 	}
 
 	function async (func) {
 		w.setTimeout(func, 0);
 	}
 
+	/**
+	 * {@link https://wcoder.github.io/notes/string-format-for-string-formating-in-javascript}
+	 * @param {string} format
+	 * @param {array} args
+	 */
+	function format (format, args) {
+		return format.replace(/\{(\d+)\}/g, function(m, n){
+			return args[n] ? args[n] : m;
+		});
+	}
+
 }(window, document));