Rob Lahaye <[EMAIL PROTECTED]> writes: | Index: src/boost.C | =================================================================== | RCS file: /cvs/lyx/lyx-devel/src/boost.C,v | retrieving revision 1.4 | diff -u -r1.4 boost.C | --- src/boost.C 2003/09/09 17:25:17 1.4 | +++ src/boost.C 2003/09/10 05:06:32 | @@ -51,7 +51,7 @@ | lyxerr << "Assertion triggered in " << function << " by \"" << | expr << " in file " << file << ":" << line << endl; | emergencyCleanup(); | - assert(false); | + BOOST_ASSERT(false); | }
This will create a citcular... Why is it needed? | -------------------------------------------------------- > | A few lines before, there is: > | void emergencyCleanup() | { | static bool didCleanup; | if (didCleanup) | return; > | didCleanup = true; > | LyX::emergencyCleanup(); | } > | Wouldn't "static bool didCleanup (false)" be better? | It worries me seeing a test of an uninitialized variable. Statics are default initialized (to zero -> false in this case) -- Lgb