On Apr 7, 2010, at 6:40 PM, Jonathan Lundell wrote: > On Apr 7, 2010, at 6:33 PM, mdipierro wrote: > >> Do you still have the patch. I remember applying one patch from you >> and rejecting one that I thought would break certain expressions. If >> you resend it I will double check. > > You applied the patch to test_template.py, which was also broken. > > I need to update it, since template.py has changed since I sent it last time. > > A problem is that we don't have a good test case. The unit test doesn't have > any embedded newlines, which is what re_strings is all about. > > Back in a minute.
This is for testing: http://web.me.com/jlundell/filechute/template.zip I do not understand one part of the pattern. For example: + r"'(?!'')(?:[^'\\]+|\\.)*'" Why do we need the |\\. ? What function is it serving? It seems to make the alternation useless. Also, I'm not sure why we're escaping the right square bracket. > > >> >> On Apr 7, 7:43 pm, Jonathan Lundell <jlund...@pobox.com> wrote: >>> On Apr 7, 2010, at 2:28 PM, Timothy Farrell wrote: >>> >>>> I've looked at this problem extensively and I'm convinced that the >>>> solution is to re-implement template.py without complicated regexps or >>>> remove the claim of Jython support. >>> >>>> The exact problem is related to the size of a view that is run through the >>>> template module. The "re_strings" as it is used in the parse() method is >>>> too complicated. A temporary solution is to comment out that line but >>>> templates will not render some Javascript properly. >>> >>> re_strings looks just plain broken to me. The inner parens should not be >>> capturing, there should be non-capturing parens around the alternation, and >>> the single-line quote patterns shouldn't have dots in the match. >>> >>> I think. >>> >>> I sent a patch for this back in November, and Timothy reported that it >>> worked. But it never got applied. >>> >>> Want to try again? >>> >>> >>> >>>> This is ultimately a bug in Java (not even Jython) but it officially has >>>> WONT-FIX status with Sun. >>> >>>> On 4/7/2010 3:34 PM, Jonathan Lundell wrote: >>> >>>>> On Apr 7, 2010, at 12:44 PM, John Cobo wrote: >>> >>>>>> I have not written any regular expressions. So far I am just trying to >>>>>> run the web2py "welcome" app. >>> >>>>>> The only reg. exp. is what creates the welcome page view. >>> >>>>> No traceback? >>> >>>>>> Thanks. >>> >>>>>>> On 7 Apr 2010 19:29, "Jonathan Lundell" <jlund...@pobox.com> wrote: >>> >>>>>>> On Apr 7, 2010, at 9:55 AM, JC11 wrote: >>> >>>>>>>> I am getting the dreaded error: >>>>>>>> RuntimeError: maximum recursion depth exceeded >>> >>>>>>>> I am using web2py vsn. 1.76.5, jython version 2.5.1 on Windows XP. >>> >>>>>>>> I have altered the web2py 'welcome' application defaault controller >>>>>>>> to: return 'Hello World' rather than return dict(message=T('Hello >>>>>>>> World')). >>>>>>>> This solved the recursion problem, but only if one does not use web2py >>>>>>>> views :( >>> >>>>>>>> I tried adding the following to the default controller, but it did not >>>>>>>> help. >>>>>>>> import sys >>>>>>>> sys.setrecursionlimit(2500) >>> >>>>>>>> Any suggestions ? >>> >>>>>>> What's the regex that's causing the problem? >>> >>>>>>>> On Apr 7, 4:16 pm, Jonathan Lundell <jlund...@pobox.com> wrote: >>>>>>>>> On Apr 7, 2010, at 7:14 AM, mdipierro wrote: >>> >>>>>>>>>> Some people have reported problems with Jython due to a bug in Java >>>>>>>>>> regex. I tried Jython2.5rc2 and it worked for me. Let us know. >>> >>>>>>>>> One or two of the regex patches a while back (URL checking IIRC) was >>>>>>>>> aimed at preventing excessive backtracking under Jython. If anyone >>>>>>>>> runs into that problem again, they should report it. I'm pretty sure >>>>>>>>> I understand how to avoid at least the problem we had then, by making >>>>>>>>> alternations mutually exclusive. >>> >>>>>>>>>> On Apr 7, 6:14 am, JC11 <john.c...@gmail.com> wrote: >>>>>>>>>>> Thanks, >>> >>>>>>>>>>> I am sure I searched in the 'book', how silly of me not to find it. >>>>>>>>>>> What about Oracle and Jython ? The last line of the entrie reads: >>>>>>>>>>> 'You will be able to use DAL('sqlite://...') and >>>>>>>>>>> DAL('postgres://...') only.' >>> >>>>>>>>>>> John C. >>> >>>>>>>>>>> On Apr 7, 11:54 am, Kuba Kucharski <kuba.kuchar...@gmail.com> wrote: >>> >>>>>>>>>>>> Hi, >>> >>>>>>>>>>>> here they are: >>> >>>>>>>>>>>> http://web2py.com/book/default/section/12/9 >>> >>>>>>>>>>>> + use newest stable >>> >>>>>>>>>>>> -- >>>>>>>>>>>> Kuba -- You received this message because you are subscribed to the Google Groups "web2py-users" group. To post to this group, send email to web...@googlegroups.com. To unsubscribe from this group, send email to web2py+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/web2py?hl=en.