Sasha Khapyorsky wrote:
> On 19:24 Thu 12 Nov , Sammy Wilborn wrote:
>>>> - printf("\nError: Lame choice!\n");
>>>> + printf("\nError: Please try again.\n");
>>> "Try again" is good, but it is useful to provide a reason too, so let's
>>> do it even more "professional":
>>>
>>> printf("\nError: Lame choice! Please try again.\n");
>> How about the following?
>>
>> printf("\nError: Invalid choice! Please try again.\n");
>
> This works too. Would you care about the patch?
I think the point is to not upset the customer by suggesting that
they are "lame". OFED, meet customer spending LOTS of money. Customer,
meet OFED! :) Do you need a patch submitted making that change?
And, thanks for the comment about using the current master branch.
I'll try to do that in the future. I'm still coming up to speed on
processes and remain mostly "git" illiterate.
Thanks,
Mike
>
> Sasha
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
