On 11/9/17 6:25 AM, Goldwyn Rodrigues wrote:
>
>
> On 11/08/2017 10:27 PM, Eric Sandeen wrote:
>> On 10/10/17 5:58 AM, Goldwyn Rodrigues wrote:
...
>> + case 'N':
>> +#ifdef HAVE_PWRITEV2
>> + pwritev2_flags |= RWF_NOWAIT;
>> +#else
>> + printf(_("Not built with pwritev2 functionality\n"));
>> +#endif
>> + break;
>>
>
> I had proposed something similar (with another message) in v2, but Dave
> did not like it. I am fine to make it work either ways. Let me know.
> Note, We would have to put similar checks for -V option which would add
> some more ifdefs, which will make it a mess.
Oh, right sorry. Dave had suggested that the default case should say:
"command -%c not supported"
to support all the configurable options, and I agree that's a better solution.
-Eric
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
