Just code style. &nbsp;Rest of project uses style:<div><br></div><div>&nbsp;&nbsp; &nbsp; } else {</div><div><br></div><div>Braces on same line.</div><div><br></div><div>Also, no need for explicit string concat here:</div><div><br></div><div>
<span class="Apple-style-span" style="border-collapse: collapse; ">&nbsp;&nbsp; &nbsp; die &quot;nothing found matching &#39;&quot; . $self-&gt;{prefix} . &quot;&#39;.\n&quot;;</span><br></div><div><span class="Apple-style-span" style="border-collapse: collapse;"><br>
</span></div><div><span class="Apple-style-span" style="border-collapse: collapse;">Just interoplate it. &nbsp;The ifs are very C-ish too. &nbsp;You could just do:</span></div><div><span class="Apple-style-span" style="border-collapse: collapse;"><br>
</span></div><div><span class="Apple-style-span" style="border-collapse: collapse;">die &quot;Nothing found matching $self-&gt;{prefix}.\n&quot; if $self-&gt;{prefix};</span></div><div><span class="Apple-style-span" style="border-collapse: collapse;">die &quot;Nothing found to restore. &nbsp;(empty .meta file?)\n&quot;;</span></div>
<div><span class="Apple-style-span" style="border-collapse: collapse;"><br></span></div><div><span class="Apple-style-span" style="border-collapse: collapse;">(also note capitalized sentences in error messages.)</span></div>
<div><br><br><div class="gmail_quote">On Mon, Nov 3, 2008 at 1:49 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;">
Here&#39;s a proposed patch to make brackup-restore complain if nothing is<br>
found to restore (probably most useful with --just, but works with --all<br>
as well.<br>
<br>
Any thoughts/comments/objections before I commit?<br>
<br>
Cheers,<br>
Gavin<br>
<br>
<br>
=== modified file &#39;lib/Brackup/Restore.pm&#39;<br>
--- lib/Brackup/Restore.pm &nbsp; &nbsp; &nbsp;2008-07-29 21:29:25 +0000<br>
+++ lib/Brackup/Restore.pm &nbsp; &nbsp; &nbsp;2008-11-01 01:48:17 +0000<br>
@@ -71,6 +71,7 @@<br>
 &nbsp; &nbsp; $self-&gt;{_target} = $target;<br>
 &nbsp; &nbsp; $self-&gt;{_meta} &nbsp; = $meta;<br>
<br>
+ &nbsp; &nbsp;my $restore_count = 0;<br>
 &nbsp; &nbsp; while (my $it = $parser-&gt;readline) {<br>
 &nbsp; &nbsp; &nbsp; &nbsp; my $type = $it-&gt;{Type} || &quot;f&quot;;<br>
 &nbsp; &nbsp; &nbsp; &nbsp; die &quot;Unknown filetype: type=$type, file: $it-&gt;{Path}&quot; unless $type =~ /^[ldfp]$/;<br>
@@ -88,6 +89,7 @@<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }<br>
 &nbsp; &nbsp; &nbsp; &nbsp; }<br>
<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;$restore_count++;<br>
 &nbsp; &nbsp; &nbsp; &nbsp; my $full = $self-&gt;{to} . &quot;/&quot; . $it-&gt;{Path};<br>
<br>
 &nbsp; &nbsp; &nbsp; &nbsp; # restore default modes from header<br>
@@ -101,10 +103,21 @@<br>
 &nbsp; &nbsp; &nbsp; &nbsp; $self-&gt;_restore_file &nbsp; &nbsp; ($full, $it) if $type eq &quot;f&quot;;<br>
 &nbsp; &nbsp; }<br>
<br>
- &nbsp; &nbsp;warn &quot; * fixing stat info\n&quot; if $self-&gt;{verbose};<br>
- &nbsp; &nbsp;$self-&gt;_exec_statinfo_updates;<br>
- &nbsp; &nbsp;warn &quot; * done\n&quot; if $self-&gt;{verbose};<br>
- &nbsp; &nbsp;return 1;<br>
+ &nbsp; &nbsp;if ($restore_count) {<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;warn &quot; * fixing stat info\n&quot; if $self-&gt;{verbose};<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;$self-&gt;_exec_statinfo_updates;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;warn &quot; * done\n&quot; if $self-&gt;{verbose};<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;return 1;<br>
+ &nbsp; &nbsp;}<br>
+<br>
+ &nbsp; &nbsp;else {<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;if ($self-&gt;{prefix}) {<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;die &quot;nothing found matching &#39;&quot; . $self-&gt;{prefix} . &quot;&#39;.\n&quot;;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;}<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;else {<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;die &quot;nothing found to restore.\n&quot;;<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;}<br>
+ &nbsp; &nbsp;}<br>
&nbsp;}<br>
<br>
&nbsp;sub _output_temp_filename {<br>
<br>
</blockquote></div><br></div>