<br><br><div class="gmail_quote">2012/8/30 Thomas Graf <span dir="ltr"><<a href="mailto:tgraf@suug.ch" target="_blank">tgraf@suug.ch</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, Aug 30, 2012 at 12:12:24AM +0600, Марк Коренберг wrote:<br>
> > Sorry, I have repaired that thing in same branch named "bugs" (now,<br>
> > rebased from your current master branch). I don't know if this patch is<br>
> > really needed (nothing change, just clean up). It will be nice If you<br>
> > include that in main repo.<br>
> ><br>
> ><br>
> > Also, new set of fixes :)<br>
><br>
> Please review. The most problem for me is to tag text correctly for<br>
> auto-generated documentation. I just copied from another place...<br>
<br>
</div>The fact that you need to fix all users within the library also<br>
means that _all_ applications would need to fix their usage of<br>
these functions.<br>
<br>
I could live with that if we actually fixed a real bug but constification<br>
is only a cleanup and not worth the work it causes.<br>
<br>
I believe it's vital for the library to try and stay as stable as possible<br>
requring as little API changes between versions as possible.<br>
<br>
Can you please drop the char ** -> const char ** patch from your pull<br>
request and rebase?<br></blockquote><div>OK, done that. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also, while I'm perfectly fine with using github the way you do, could<br>
you, going forward, please send a formal pull requestincluding each patch<br>
in a separate email such as generated by git-send-mail. I'm reviewing<br>
many patches every day and email is just the easiest option and it gives<br>
others on the list to do reviews easily as well.<br></blockquote><div>Ok, next time I will do that your way. Should I make pull request in future as well?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Thanks!<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Segmentation fault<br>