<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, May 4, 2018 at 8:46 AM, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, 4 May 2018 07:55:10 -0500<br>
Matt Hoosier <<a href="mailto:matt.hoosier@gmail.com">matt.hoosier@gmail.com</a>> wrote:<br>
<br>
> On Fri, May 4, 2018 at 2:13 AM, Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com">ppaalanen@gmail.com</a>> wrote:<br>
> <br>
</span><span class="">> > The crucial call to make is weston_log_set_handler() with non-NULL<br>
> > arguments before any other libweston calls.<br>
> >  <br>
> <br>
> I wonder if this could be made clearer by adding an assert in weston_log()<br>
> if no handler has been installed yet.<br>
<br>
</span>Definitely. :-)<br>
A shout to stderr and abort() would be fine. Or even have the handlers<br>
initilized to ones that shout and abort().<br>
<br>
<br>
Thanks,<br>
pq<br>
</blockquote></div><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Done and published on the list for comments.</div></div>