Andrew Dunstan <[EMAIL PROTECTED]> writes:
I see that psql/mainloop.c contains this comment:
* FIXME: rewrite this whole thing with flex
Is now the time to do that?
If you feel like giving it a shot, ain't nobody gonna stand in your way. MainLoop() is well past the level of unmaintainability IMHO (which is why I'd not tried to touch it in the original quick patch).
I'm not totally convinced that flex can do the job, but in any case a ground-up redesign for legibility might be the only cure.
I will take a look - no guarantees though. I agree that flex might well end up making things more obscure than less.
Keeping track of open quote operators to make sure we have proper nesting and closing looks like it might be a bit messy, but doable.
cheers
andrew
---------------------------(end of broadcast)--------------------------- TIP 3: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly
