$.tablesorter: remove and inline getTextFromRowAndCellIndex()

It introduced unnecessary indirection and was only called once.

Change-Id: I519cc1f04a581d41aa23229182c55466ed7757c8

diff --git a/resources/jquery/jquery.tablesorter.js b/resources/jquery/jquery.tablesorter.js
index 8320304..e252ba5 100644 (file)
--- a/resources/jquery/jquery.tablesorter.js
+++ b/resources/jquery/jquery.tablesorter.js
@@ -113,14 +113,6 @@
                }
        }
 
-       function getTextFromRowAndCellIndex( rows, rowIndex, cellIndex ) {
-               if ( rows[rowIndex] && rows[rowIndex].cells[cellIndex] ) {
-                       return $.trim( getElementSortKey( rows[rowIndex].cells[cellIndex] ) );
-               } else {
-                       return '';
-               }
-       }
-
        function detectParserForColumn( table, rows, cellIndex ) {
                var l = parsers.length,
                        nodeValue,
@@ -130,8 +122,13 @@
                        concurrent = 0,
                        needed = ( rows.length > 4 ) ? 5 : rows.length;
 
-               while( i < l ) {
-                       nodeValue = getTextFromRowAndCellIndex( rows, rowIndex, cellIndex );
+               while ( i < l ) {
+                       if ( rows[rowIndex] && rows[rowIndex].cells[cellIndex] ) {
+                               nodeValue = $.trim( getElementSortKey( rows[rowIndex].cells[cellIndex] ) );
+                       } else {
+                               nodeValue = '';
+                       }
+
                        if ( nodeValue !== '') {
                                if ( parsers[i].is( nodeValue, table ) ) {
                                        concurrent++;