[haiku-development] Re: Review; JSON Parser Changes

  • From: Andrew Lindesay <apl@xxxxxxxxxxxxxx>
  • To: haiku-development@xxxxxxxxxxxxx
  • Date: Fri, 28 Apr 2017 23:52:13 +1200

Hello Axel;

And why a flex-based parser? What was wrong with the old one?
While I'm not a fan of flex myself (I like to write parsers manually, 

I was most tempted to code it by hand actually, but my reckoning was
that is _probably_ safer to implement the streaming with flex to start
with.  I would like to see it eventually hand-written again just because
the C - C++ interfacing I had to do around flex and gcc2 is "not
lovely".  Something for the future.

BTW Andrew, I noticed a number of minor coding style violations in your 
diff; I point out a few for you below:

Oh dear!  thank you.  I'll work through that list...

Regards.

-- 
Andrew Lindesay

Other related posts: