<br><br><div class="gmail_quote">On Tue, Nov 4, 2008 at 1:27 AM, Gavin Carr <span dir="ltr">&lt;<a href="mailto:gavin@openfusion.com.au">gavin@openfusion.com.au</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="Ih2E3d">On Mon, Nov 03, 2008 at 10:08:04PM -0800, Brad Fitzpatrick wrote:<br>
&gt; Just code style. &nbsp;Rest of project uses style:<br>
&gt; &nbsp; &nbsp; &nbsp;} else {<br>
&gt;<br>
&gt; Braces on same line.<br>
&gt;<br>
&gt; Also, no need for explicit string concat here:<br>
&gt;<br>
&gt; &nbsp; &nbsp; &nbsp;die &quot;nothing found matching &#39;&quot; . $self-&gt;{prefix} . &quot;&#39;.\n&quot;;<br>
&gt;<br>
&gt; Just interoplate it. &nbsp;The ifs are very C-ish too. &nbsp;You could just do:<br>
&gt;<br>
&gt; die &quot;Nothing found matching $self-&gt;{prefix}.\n&quot; if $self-&gt;{prefix};<br>
&gt; die &quot;Nothing found to restore. &nbsp;(empty .meta file?)\n&quot;;<br>
<br>
</div>Cool, all adjusted.<br>
<div class="Ih2E3d"><br>
&gt; (also note capitalized sentences in error messages.)<br>
<br>
</div>Yep, although the dies here are caught and rethrown by brackup-restore,<br>
and I thought the lowercase version looked slightly better:<br>
<br>
 &nbsp;Error doing restore: nothing found matching &#39;foobar&#39;.<br>
<br>
vs.<br>
<br>
 &nbsp;Error doing restore: Nothing found matching &#39;foobar&#39;.<br>
<br>
I don&#39;t really care though - which do you prefer?</blockquote><div><br></div><div>Oh, the former&#39;s nice. &nbsp;Either way.</div><div><br></div><div><br></div></div>