change wording of ok()-messages in winspool-tests?

Alexandre Julliard julliard at winehq.org
Mon Jun 25 02:58:55 CDT 2012


Julian Rüger <jr98 at gmx.net> writes:

> Hi Francois and list,
>
> [/nitpick-mode on]
> I'm preparing a patch to fix typos/spelling/grammar in
> dlls/winspool.drv/tests/info.c, where I stumbled across repeated
> messages like
>
>     ok( !retval, "function result wrong! False expected\n");
> or
>     ok( size == exact, "Parameter size wrong! %d expected got %d\n",
>         exact, size);
>
>
> Do you think it's worth changing these?
> (I think "Wrong function result: 'false' expected\n", and "Wrong
> parameter size: expected %d, got %d\n" would be better)
>
> Do we have guidelines for strings like these?
> For example: start with a captial letter, no punctuation at the end...
>
> What tests have "good" style I can use as reference?
>
> Best regards,
> Julian
>
> PS: Am I bike shedding here? Does anyone care about stuff like this?

Test failure messages are not supposed to be seen, if they are seen it's
a bug. So it's not worth spending effort making them look nice or
consistent.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list