[m-rev.] Re: for review: parsing_utils improvements

Ralph Becket rafe at csse.unimelb.edu.au
Fri Oct 2 09:15:30 AEST 2009


Ian MacLarty, Thursday,  1 October 2009:
> On Thu, Oct 1, 2009 at 1:49 PM, Ralph Becket <rafe at csse.unimelb.edu.au> wrote:
> > Ian MacLarty, Wednesday, 30 September 2009:
> >> On Wed, Sep 30, 2009 at 12:12 PM, Ian MacLarty
> >> <maclarty at csse.unimelb.edu.au> wrote:
> >> > Here are some benchmarks I did with the sparql parser with an input of
> >> > 4.8M (times are an average of 3 runs):
> >>
> >> Correction: 48M (48 100 031 bytes).
> >
> > Sorry, I thought I'd replied to this before: you've convinced me this is
> > a good idea!
> 
> Cool!  Any call on whether I should use a flag in the src or not?  It
> seems to speed things up a bit in the case where the mutables are not
> updated, but I can't think of many scenarios where you're not going to
> want an error context (maybe if you're parsing a lot of computer
> generated data, but even then an error context would be useful if
> there was an error).

I think the flag adds complexity for little benefit - I'd skip it.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list