[PATCH] tst-longjmp_chk2: add comments/sanity check

Joseph S. Myers joseph@codesourcery.com
Mon Dec 30 15:20:00 GMT 2013


On Sun, 29 Dec 2013, Mike Frysinger wrote:

> +  /* Sanity check to keep test from looping forever (in case the longjmp
> +     chk code is slightly broken).  */
> +  assert (pass++ < 5);

Side effects inside an argument to assert don't seem a particularly good 
idea, even if we don't expect tests to use -DNDEBUG....

-- 
Joseph S. Myers
joseph@codesourcery.com



More information about the Libc-alpha mailing list