<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Aug 23, 2013, at 3:42 PM, Jordan Rose <<a href="mailto:jordan_rose@apple.com">jordan_rose@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br>On Aug 23, 2013, at 15:40 , Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:<br><br><blockquote type="cite">On Tue, Aug 20, 2013 at 10:00 AM, Jordan Rose <<a href="mailto:jordan_rose@apple.com">jordan_rose@apple.com</a>> wrote:<br><blockquote type="cite"><br>On Aug 20, 2013, at 9:50 , Stephen Kelly <<a href="mailto:steveire@gmail.com">steveire@gmail.com</a>> wrote:<br><br><blockquote type="cite">Jordan Rose wrote:<br><blockquote type="cite">Obviously, the best case is to get them to start using<br>__attribute__((noreturn)) and/or __attribute__((analyzer_noreturn)). I can<br>see adding qtFatal to the list, though, since Qt is a fairly widespread<br>toolkit.<br></blockquote><br>qt_assert is already __attribute__((noreturn)).<br><br>#ifndef Q_CC_MSVC<br>Q_NORETURN<br>#endif<br>Q_CORE_EXPORT void qt_assert(const char *assertion, const char *file, int<br>line) Q_DECL_NOTHROW;<br></blockquote><br>Then...why isn't it working? Dmitri, if you have a test case, please file a Bugzilla report, I guess.<br></blockquote><br>OK, I found why. Because the noreturn attribute was added to the<br>declaration only in Qt 5. In Qt 4, which will be still relevant for<br>at least a few years from now, qt_assert is not marked with noreturn:<br><br><a href="http://qt.gitorious.org/qt/qt/blobs/4.8/src/corelib/global/qglobal.h#line1815">http://qt.gitorious.org/qt/qt/blobs/4.8/src/corelib/global/qglobal.h#line1815</a><br></blockquote><br>Okay, seems reasonable to me.<br></div></blockquote><div><br></div>I am not very keen on modifying the analyzer code to support old versions of Qt...</div><div><br></div><div>(If you were to do this, please put in a comment stating that that is temporary and should be removed. )</div><div><br><blockquote type="cite" dir="auto"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br>Jordan<br>_______________________________________________<br>cfe-dev mailing list<br><a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a></div></blockquote></div><br></body></html>