On 11/07/2013 10:01 PM, Bernhard Voelker wrote:
> On 11/07/2013 08:44 PM, Pádraig Brady wrote:
>> Hopefully the attached addresses this issue.
>
> Hi Pádraig,
>
>> diff --git a/THANKS.in b/THANKS.in
>> index 891b376..6588376 100644
>> --- a/THANKS.in
>> +++ b/THANKS.in
> ...
>> @@ -9559,12 +9559,25 @@ the whole file. @var{bytes} can be followed by a
>> size specification like
>> +requiring a sync for every character in every file. This can become
>
> minor nit: s/\. /. /
> http://xkcd.com/1285/ ;-)
Hah excellent :)
> The short option doesn't accept the remove methods:
>
> $ src/shred -u unlink x
> src/shred: unlink: failed to open for writing: No such file or directory
>
> Was this intentional?
Yes, optional args to short options have many issues.
thanks for the review!
Pádraig.
