> Not necessarily relevant but marpa_v_step() can return a negative > number on failure, and I think this is not handled. In particular > there are a number of cases of unexpected returns from > marpa_v_step() which you handle via fall-through which probably are > better treated as fatal errors.
> Same with marpa_v_new() -- it can return NULL on failure and this is > not checked for. I have no real evidence that this has anything to > do with your problem, but this is C language so subtle stuff could > be happening with memory overwrites, etc. Agreed. > And the checks *do* belong in your final code, so you might as well > have them in for debugging. Agreed as well. Will definitely put in code to handle failures ... Oh, the bocage wrapper needs them as well. If I remember correctly it does not handle these things either. Are the negative numbers from marpa_v_step() regular marpa error codes, or are they their own set ? Memory. Right. While I am mostly confident that I managed to avoid issues, due to the lot of assertions I put in all over the place it will definitely not hurt (except speed (*) of the system) to activate the memory debugging Tcl's memory utility functions provide, and should trip anything bad I may have in the code. Thank you for the reminder of that. (*) Especially if I have it check the entire set of allocations on each and every malloc, realloc, or free, instead of just the specific memory allocated or freed by the current call. > > On Thu, Aug 17, 2017 at 6:55 PM, Andreas Kupries <akupr...@shaw.ca> wrote: > > > >> > >> > Looked some more at `rule` and `rslot` and I not 100% sure it's wrong, > >> but > >> > I can't convince myself it's right. -- See you, Andreas Kupries <akupr...@shaw.ca> <http://core.tcl.tk/akupries/> Developer @ SUSE (MicroFocus Canada LLC) <andreas.kupr...@suse.com> Tcl'2017, Oct 16-20, Houston, TX, USA. http://www.tcl.tk/community/tcl2017/ EuroTcl 2017, Jul 8-9, Berlin/DE, http://www.eurotcl.tcl3d.org/ ------------------------------------------------------------------------------- -- You received this message because you are subscribed to the Google Groups "marpa parser" group. To unsubscribe from this group and stop receiving emails from it, send an email to marpa-parser+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.