On 04/05/2013 11:31 PM, Scott Kostyshak wrote:
Should LyX exit more gracefully here?

$ lyx -e pdf2 Intro.lyx
Missing user LyX directory
----------------------------------------
You have specified a non-existent user LyX directory,
/home/scott/lyxbuilds/master/user-dir.
It is needed to keep your own configuration.
Assuming answer is &Exit LyX

Note that this only happens in devel mode. In release mode, we continue and exit in the usual way. So I think this one is fine, though maybe we don't really need to assert in this case. You might could check some other flag?

There's actually a general need to look through all the LASSERT statements from this point of view. In many cases, we just continue in release mode if the assertion fails, when that is clearly going to lead to a crash, and we could take some other remedial action. Often, just returning out of that function will do.

Richard

No user LyX directory. Exiting.
support/lassert.cpp (35): ASSERTION pimpl_->application_.get()
VIOLATED IN /home/scott/lyxbuilds/master/build/src/LyX.cpp:439
Assertion triggered in void lyx::doAssert(const char*, const char*,
long int) by failing check "false" in file
/home/scott/lyxbuilds/master/build/src/support/lassert.cpp:37
Aborted (core dumped)
$

Reply via email to