<div dir="ltr"><div><div>I just grabbed a fresh copy of sword via svn.<br><br>On Sat, Jan 19, 2013 at 11:13 AM, Troy A. Griffitts <span dir="ltr"><<a href="mailto:scribe@crosswire.org" target="_blank">scribe@crosswire.org</a>></span> wrote:<br>
In
anticipation of a new release, I have a large checkin I'm about to
commit which will break everything for everyone compiling with -Werror.<br><br></div>The Makefile that comes with sword uses the -Werror directive, so grabbing via svn and making is currently broken on the making of .libs/swmodule.o.<br>
<br>Could it be fixed it so that what comes via svn isn't broken. Is the solution to remove -Werror and try again?<br><br></div>~A<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 21, 2013 at 6:49 AM, Troy A. Griffitts <span dir="ltr"><<a href="mailto:scribe@crosswire.org" target="_blank">scribe@crosswire.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 01/21/2013 12:28 PM, Nic Carter wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Is there a reason for this being the abbreviation and not "setIntroductions(bool)" and "isIntroductions()"? I find it strange that you've fixed up "Lang()" and introduced "Intros()"... But, thanks for this change. It was confusing before! :)<br>
</blockquote>
<br></div>
Yeah, well. :) Here was my thinking on this. No one says "Lang" as a spoken abbreviation for Language, but we all use "Intros" regularly. In fact-- seeking to justify my behavior-- it looks like google even claims "intro" as it's own word: <a href="https://www.google.com/search?q=definition+intro" target="_blank">https://www.google.com/search?<u></u>q=definition+intro</a><div class="im">
<br>
<br>
:)<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, I think you forgot a "const" in echomod.cpp? (one of the tests.) I say "think" cause it threw an error for me and then I realised I didn't need that in my stuff, so I simply deleted it rather than "fix" it..<br>
</blockquote>
<br></div>
Fixed. Thanks for the heads up Nic.<span class="HOEnZb"><font color="#888888"><br>
<br>
Troy</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
. ;) Thanks for all of this. :) Still double checking it all with my stuff, but looking good so far :) Thanks, ybic nic... :) ______________________________<u></u>_________________ sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org" target="_blank">sword-devel@crosswire.org</a> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/<u></u>mailman/listinfo/sword-devel</a> Instructions to unsubscribe/change your settings at above page <br>
</blockquote>
<br>
<br>
______________________________<u></u>_________________<br>
sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org" target="_blank">sword-devel@crosswire.org</a><br>
<a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/<u></u>mailman/listinfo/sword-devel</a><br>
Instructions to unsubscribe/change your settings at above page<br>
</div></div></blockquote></div><br></div>