<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">I’m behind ditching the function identifier #defines but not their text names.<div class=""><br class=""></div><div class=""><br class=""></div><div class="">The varargs based error function seems like a good idea too.  It would be nicer if the ERR_raise_error function included the file and line information without them needing to be specified.  As an exercise for the reader, I’ll skip any implementation details but will note that it is possible without using variadic macros.</div><div class=""><br class=""></div><div class=""><br class=""></div><div class="">Pauli<br class=""><div class=""><br class="webkit-block-placeholder"></div><div class=""><br class=""></div><div class=""><span style="font-size: 9px;" class="">Hint:</span></div><div class=""><blockquote style="margin: 0px 0px 0px 40px; border: none; padding: 0px;" class=""><div class=""><font face="Courier New" style="font-size: 9px;" class="">#define ERR_raise_error ERR_raise_error_internal(__FILE__, __LINE__, __FUNC__)</font></div><div class=""><font face="Courier New" style="font-size: 9px;" class="">int (*)(int, const char *, ...) ERR_raise_error_internal(const char *, int, const char *);</font></div></blockquote></div><div class="">
<div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;">-- <br class="">Dr Paul Dale | Cryptographer | Network Security & Encryption <br class="">Phone +61 7 3031 7217<br class="">Oracle Australia</div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;"><br class=""></div><br class="Apple-interchange-newline">
</div>
<div><br class=""><blockquote type="cite" class=""><div class="">On 14 Jun 2019, at 12:04 pm, Viktor Dukhovni <<a href="mailto:openssl-users@dukhovni.org" class="">openssl-users@dukhovni.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">On Wed, Jun 12, 2019 at 05:51:44AM +0200, Richard Levitte wrote:<br class=""><br class=""><blockquote type="cite" class="">A discussion point in that PR is whether it's still interesting to<br class="">keep information on the system function/callback that was called when<br class="">we're reporting a system error, i.e. this type of error report:<br class=""><br class="">    SYSerr(SYS_F_FSTAT, errno);<br class=""><br class="">(incidently, there's another PR, 9072, which changes those to<br class="">'SYSerr("fstat", errno)')<br class=""><br class="">So, the main points of discussion seem to be:<br class=""><br class="">- should we remove function indicators in our error reporting?<br class="">- should we make an exception for system errors (SYSerr())?<br class=""></blockquote><br class="">Bottom line, my take is that function indicators should stay, but<br class="">string names would be more convenient all around than numeric codes.<br class=""><br class="">-- <br class=""><span class="Apple-tab-span" style="white-space:pre">       </span>Viktor.<br class=""></div></div></blockquote></div><br class=""></div></body></html>