That's a great set of patches - thanks! The only minor nit that I see
is that you do:
jQuery.ajaxSetup({ ifModified: true });In your tests instead of putting the setting inline in the $.ajax() call (which would probably be preferred, since it'll be less likely to affect other tests). I made the change and committed the files: http://dev.jquery.com/changeset/6388 Thanks again! --John On Mon, Jun 15, 2009 at 3:53 AM, lawrence.pit<lawrence....@gmail.com> wrote: > > Hi All, > > I've created a patch + tests that improves support for the Last- > Modified header and adds Etag support. I'd appreciate it if some could > take a look at this. > > http://dev.jquery.com/ticket/4764 > > > > Thanks, > Lawrence > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "jQuery Development" group. To post to this group, send email to jquery-dev@googlegroups.com To unsubscribe from this group, send email to jquery-dev+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/jquery-dev?hl=en -~----------~----~----~----~------~----~------~--~---
