Bug Tracker

Ticket #2543: vars.txt

File vars.txt, 10.2 KB (added by flesler, 14 years ago)

patch.txt is wrong, check any of these

Line 
1Index: src/core.js
2===================================================================
3--- src/core.js (revision 5084)
4+++ src/core.js (working copy)
5@@ -729,24 +729,25 @@
6 
7        // args is for internal usage only
8        each: function( object, callback, args ) {
9+               var i = 0, name, length = object.length;
10                if ( args ) {
11-                       if ( object.length == undefined ) {
12-                               for ( var name in object )
13+                       if ( length == undefined ) {
14+                               for ( name in object )
15                                        if ( callback.apply( object[ name ], args ) === false )
16                                                break;
17                        } else
18-                               for ( var i = 0, length = object.length; i < length; i++ )
19+                               for ( ; i < length; i++ )
20                                        if ( callback.apply( object[ i ], args ) === false )
21                                                break;
22 
23                // A special, fast, case for the most common use of each
24                } else {
25-                       if ( object.length == undefined ) {
26-                               for ( var name in object )
27+                       if ( length == undefined ) {
28+                               for ( name in object )
29                                        if ( callback.call( object[ name ], name, object[ name ] ) === false )
30                                                break;
31                        } else
32-                               for ( var i = 0, length = object.length, value = object[0];
33+                               for ( var value = object[0];
34                                        i < length && callback.call( value, i, value ) !== false; value = object[++i] ){}
35                }
36 
37@@ -791,9 +792,9 @@
38 
39        // A method for quickly swapping in/out CSS properties to get correct calculations
40        swap: function( elem, options, callback ) {
41-               var old = {};
42+               var old = {}, name;
43                // Remember the old values, and insert the new ones
44-               for ( var name in options ) {
45+               for ( name in options ) {
46                        old[ name ] = elem.style[ name ];
47                        elem.style[ name ] = options[ name ];
48                }
49@@ -801,7 +802,7 @@
50                callback.call( elem );
51 
52                // Revert the old values
53-               for ( var name in options )
54+               for ( name in options )
55                        elem.style[ name ] = old[ name ];
56        },
57 
58@@ -831,7 +832,7 @@
59        },
60 
61        curCSS: function( elem, name, force ) {
62-               var ret;
63+               var ret, estyle = elem.style;
64 
65                // A helper method for determining if an element's values are broken
66                function color( elem ) {
67@@ -844,7 +845,7 @@
68 
69                // We need to handle opacity special in IE
70                if ( name == "opacity" && jQuery.browser.msie ) {
71-                       ret = jQuery.attr( elem.style, "opacity" );
72+                       ret = jQuery.attr( estyle, "opacity" );
73 
74                        return ret == "" ?
75                                "1" :
76@@ -852,17 +853,17 @@
77                }
78                // Opera sometimes will give the wrong display answer, this fixes it, see #2037
79                if ( jQuery.browser.opera && name == "display" ) {
80-                       var save = elem.style.outline;
81-                       elem.style.outline = "0 solid black";
82-                       elem.style.outline = save;
83+                       var save = estyle.outline;
84+                       estyle.outline = "0 solid black";
85+                       estyle.outline = save;
86                }
87               
88                // Make sure we're using the right name for getting the float value
89                if ( name.match( /float/i ) )
90                        name = styleFloat;
91 
92-               if ( !force && elem.style && elem.style[ name ] )
93-                       ret = elem.style[ name ];
94+               if ( !force && estyle && estyle[ name ] )
95+                       ret = style[ name ];
96 
97                else if ( document.defaultView && document.defaultView.getComputedStyle ) {
98 
99@@ -880,7 +881,7 @@
100                        // If the element isn't reporting its values properly in Safari
101                        // then some display: none elements are involved
102                        else {
103-                               var swap = [], stack = [];
104+                               var swap = [], stack = [], i = 0;
105 
106                                // Locate all of the parent display: none elements
107                                for ( var a = elem; a && color(a); a = a.parentNode )
108@@ -888,7 +889,7 @@
109 
110                                // Go through and make them visible, but in reverse
111                                // (It would be better if we knew the exact display type that they had)
112-                               for ( var i = 0; i < stack.length; i++ )
113+                               for ( ; i < stack.length; i++ )
114                                        if ( color( stack[ i ] ) ) {
115                                                swap[ i ] = stack[ i ].style.display;
116                                                stack[ i ].style.display = "block";
117@@ -901,7 +902,7 @@
118                                        ( getComputedStyle && getComputedStyle.getPropertyValue( name ) ) || "";
119 
120                                // Finally, revert the display styles back
121-                               for ( var i = 0; i < swap.length; i++ )
122+                               for ( ; i < swap.length; i++ )
123                                        if ( swap[ i ] != null )
124                                                stack[ i ].style.display = swap[ i ];
125                        }
126@@ -924,15 +925,15 @@
127                        // but a number that has a weird ending, we need to convert it to pixels
128                        if ( !/^\d+(px)?$/i.test( ret ) && /^\d/.test( ret ) ) {
129                                // Remember the original values
130-                               var style = elem.style.left, runtimeStyle = elem.runtimeStyle.left;
131+                               var style = estyle.left, runtimeStyle = elem.runtimeStyle.left;
132 
133                                // Put in the new values to get a computed value out
134                                elem.runtimeStyle.left = elem.currentStyle.left;
135-                               elem.style.left = ret || 0;
136-                               ret = elem.style.pixelLeft + "px";
137+                               estyle.left = ret || 0;
138+                               ret = estyle.pixelLeft + "px";
139 
140                                // Revert the changed values
141-                               elem.style.left = style;
142+                               estyle.left = style;
143                                elem.runtimeStyle.left = runtimeStyle;
144                        }
145                }
146@@ -1144,13 +1145,14 @@
147 
148                // Also, we need to make sure that the correct elements are being returned
149                // (IE returns comment nodes in a '*' query)
150+               var i = 0;
151                if ( jQuery.browser.msie ) {
152-                       for ( var i = 0; second[ i ]; i++ )
153+                       for (; second[ i ]; i++ )
154                                if ( second[ i ].nodeType != 8 )
155                                        first.push( second[ i ] );
156 
157                } else
158-                       for ( var i = 0; second[ i ]; i++ )
159+                       for (; second[ i ]; i++ )
160                                first.push( second[ i ] );
161 
162                return first;
163Index: src/fx.js
164===================================================================
165--- src/fx.js   (revision 5084)
166+++ src/fx.js   (working copy)
167@@ -76,8 +76,8 @@
168                        if ( this.nodeType != 1)
169                                return false;
170 
171-                       var opt = jQuery.extend({}, optall);
172-                       var hidden = jQuery(this).is(":hidden"), self = this;
173+                       var opt = jQuery.extend({}, optall),
174+                               hidden = jQuery(this).is(":hidden"), self = this;
175                       
176                        for ( var p in prop ) {
177                                if ( prop[p] == "hide" && hidden || prop[p] == "show" && !hidden )
178Index: src/selector.js
179===================================================================
180--- src/selector.js     (revision 5084)
181+++ src/selector.js     (working copy)
182@@ -114,12 +114,12 @@
183 
184                        t = jQuery.trim(t);
185 
186-                       var foundToken = false;
187+                       var foundToken = false,
188 
189                        // An attempt at speeding up child selectors that
190                        // point to a specific element tag
191-                       var re = quickChild;
192-                       var m = re.exec(t);
193+                               re = quickChild,
194+                               m = re.exec(t);
195 
196                        if ( m ) {
197                                nodeName = m[1].toUpperCase();
198@@ -188,8 +188,8 @@
199 
200                                } else {
201                                        // Optimize for the case nodeName#idName
202-                                       var re2 = quickID;
203-                                       var m = re2.exec(t);
204+                                       re2 = quickID;
205+                                       m = re2.exec(t);
206                                       
207                                        // Re-organize the results, so that they're consistent
208                                        if ( m ) {
209@@ -222,7 +222,7 @@
210                                                ret = r = oid && (!m[3] || jQuery.nodeName(oid, m[3])) ? [oid] : [];
211                                        } else {
212                                                // We need to find all descendant elements
213-                                               for ( var i = 0; ret[i]; i++ ) {
214+                                               for ( i = 0; ret[i]; i++ ) {
215                                                        // Grab the tag name being searched for
216                                                        var tag = m[1] == "#" && m[3] ? m[3] : m[1] != "" || m[0] == "" ? "*" : m[2];
217 
218@@ -242,7 +242,7 @@
219                                                        var tmp = [];
220 
221                                                        // Try to find the element with the ID
222-                                                       for ( var i = 0; r[i]; i++ )
223+                                                       for ( i = 0; r[i]; i++ )
224                                                                if ( r[i].getAttribute("id") == m[2] ) {
225                                                                        tmp = [ r[i] ];
226                                                                        break;
227@@ -301,9 +301,9 @@
228                while ( t && t != last ) {
229                        last = t;
230 
231-                       var p = jQuery.parse, m;
232+                       var p = jQuery.parse, m, i = 0, tmp = [];
233 
234-                       for ( var i = 0; p[i]; i++ ) {
235+                       for ( ; p[i]; i++ ) {
236                                m = p[i].exec( t );
237 
238                                if ( m ) {
239@@ -331,9 +331,9 @@
240                                r = jQuery.classFilter(r, m[2], not);
241 
242                        else if ( m[1] == "[" ) {
243-                               var tmp = [], type = m[3];
244+                               var type = m[3], i = 0, rl = r.length;
245                               
246-                               for ( var i = 0, rl = r.length; i < rl; i++ ) {
247+                               for ( ; i < rl; i++ ) {
248                                        var a = r[i], z = a[ jQuery.props[m[2]] || m[2] ];
249                                       
250                                        if ( z == null || /href|src|selected/.test(m[2]) )
251@@ -352,16 +352,17 @@
252 
253                        // We can get a speed boost by handling nth-child here
254                        } else if ( m[1] == ":" && m[2] == "nth-child" ) {
255-                               var merge = {}, tmp = [],
256+                               var merge = {},
257                                        // parse equations like 'even', 'odd', '5', '2n', '3n+2', '4n-1', '-n+6'
258                                        test = /(-?)(\d*)n((?:\+|-)?\d*)/.exec(
259                                                m[3] == "even" && "2n" || m[3] == "odd" && "2n+1" ||
260                                                !/\D/.test(m[3]) && "0n+" + m[3] || m[3]),
261                                        // calculate the numbers (first)n+(last) including if they are negative
262-                                       first = (test[1] + (test[2] || 1)) - 0, last = test[3] - 0;
263+                                       first = (test[1] + (test[2] || 1)) - 0;
264+                               last = test[3] - 0;
265 
266                                // loop through all the elements left in the jQuery object
267-                               for ( var i = 0, rl = r.length; i < rl; i++ ) {
268+                               for ( i = 0, rl = r.length; i < rl; i++ ) {
269                                        var node = r[i], parentNode = node.parentNode, id = jQuery.data(parentNode);
270 
271                                        if ( !merge[id] ) {
272@@ -410,8 +411,8 @@
273        },
274 
275        dir: function( elem, dir ){
276-               var matched = [];
277-               var cur = elem[dir];
278+               var matched = [],
279+                       cur = elem[dir];
280                while ( cur && cur != document ) {
281                        if ( cur.nodeType == 1 )
282                                matched.push( cur );
283Index: src/event.js
284===================================================================
285--- src/event.js        (revision 5084)
286+++ src/event.js        (working copy)
287@@ -105,7 +105,7 @@
288                if ( elem.nodeType == 3 || elem.nodeType == 8 )
289                        return;
290 
291-               var events = jQuery.data(elem, "events"), ret, index;
292+               var events = jQuery.data(elem, "events"), ret;
293 
294                if ( events ) {
295                        // Unbind all events for the element
296Index: src/ajax.js
297===================================================================
298--- src/ajax.js (revision 5084)
299+++ src/ajax.js (working copy)
300@@ -233,8 +233,8 @@
301                // If we're requesting a remote document
302                // and trying to load JSON or Script with a GET
303                if ( (!s.url.indexOf("http") || !s.url.indexOf("//")) && s.dataType == "script" && s.type.toLowerCase() == "get" ) {
304-                       var head = document.getElementsByTagName("head")[0];
305-                       var script = document.createElement("script");
306+                       var head = document.getElementsByTagName("head")[0],
307+                               script = document.createElement("script");
308                        script.src = s.url;
309                        if (s.scriptCharset)
310                                script.charset = s.scriptCharset;
311@@ -443,9 +443,9 @@
312        },
313 
314        httpData: function( r, type ) {
315-               var ct = r.getResponseHeader("content-type");
316-               var xml = type == "xml" || !type && ct && ct.indexOf("xml") >= 0;
317-               var data = xml ? r.responseXML : r.responseText;
318+               var ct = r.getResponseHeader("content-type"),
319+                       xml = type == "xml" || !type && ct && ct.indexOf("xml") >= 0,
320+                       data = xml ? r.responseXML : r.responseText;
321 
322                if ( xml && data.documentElement.tagName == "parsererror" )
323                        throw "parsererror";