Thank you.

However, a part of the patch seems not to be applied.

The error still occurs because the correction of the following places
of event.js is not reflected.

-       // A fallback to window.onload, that will always work
-       jQuery.event.add( window, "load", jQuery.ready );
-       
-}
+};  <---- This line

It is possible to correct it with the following patches.
Could you confirm it again?

[EMAIL PROTECTED]:~/src/jquery/jquery> svn diff
Index: event/event.js
===================================================================
--- event/event.js      (revision 175)
+++ event/event.js      (working copy)
@@ -1662,4 +1662,4 @@
                jQuery.event.add( window, "load", jQuery.ready );
        }

-}
+};


And, build.sh seems to put out the error.
Should I execute gen.pl by the directory of docs?
Please correct it as follows if it is correct.

[EMAIL PROTECTED]:~/src/jquery/jquery> svn diff build.sh
Index: build.sh
===================================================================
--- build.sh    (revision 175)
+++ build.sh    (working copy)
@@ -1,3 +1,3 @@
 #!/bin/sh
 cat jquery/jquery.js event/event.js fx/fx.js ajax/ajax.js > jquery-svn.js
-cd docs && ./build.sh && cd ..
+cd docs && perl ./gen.pl && cd ..

ーーーー
Taku Sano


On 8/13/06, John Resig <[EMAIL PROTECTED]> wrote:
> Taku -
>
> Thanks for the patch! I applied it and it's in SVN rev 175. Thanks for
> your help.
>
> Also - nice to see another Perl programmer using jQuery :-)
>
> --John
>
> > I united SVN rev.174 with build.sh.
> > And, the error occurs when it is compressed with jsPacker.pl.
> >
> > I send the patch that corrects this problem.
> >
> > The patch contains the correction of the memory leak.
> > (I had sent it to ML before. )
> >
> > http://pepper.sherry.jp/jquery/svn174.patch
> >
> > [uncompress with patch] http://pepper.sherry.jp/jquery/jquery-svn174.js
> > [  compress with patch] 
> > http://pepper.sherry.jp/jquery/jquery-svn174-compress.js

_______________________________________________
jQuery mailing list
[email protected]
http://jquery.com/discuss/

Reply via email to