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

Reply via email to