https://bz.apache.org/bugzilla/show_bug.cgi?id=60903
Christophe JAILLET <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|NEW |RESOLVED --- Comment #1 from Christophe JAILLET <[email protected]> --- 2 of the fixes in your patch are already fixed, see bug 60086 fir details. The 2 other fixes in your patch won't. The removal of the 'if (script_err)...' is not an issue. The compiler should fix it by itself. The 'assert()' is no more. Moreover, moving it this way would have broken our C89 compliancy. Anyway, thanks for the patch and report. Should you find other similar issues, please report them. Static code checker can greatly help in finding bugs. *** This bug has been marked as a duplicate of bug 60086 *** -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
