Thread (3 messages) 3 messages, 2 authors, 2024-07-31

Re: [PATCH] t-example-decorate: remove test messages

From: René Scharfe <hidden>
Date: 2024-07-31 09:21:21

Am 31.07.24 um 03:41 schrieb Ghanshyam Thakkar:
If we want to further simplify, I don't see any need for having 'ret'
either and to just call the methods in check():

    check(add_decoration(&vars->n, vars->two, NULL), ==, NULL);

which would also provide more context in the stdout rather than printing
'check(ret == NULL)'. But, I believe you would have already considered
that but kept 'ret' in favor of code readability, so I am also fine with
it. Thanks for patch.
Good idea!  Worth a separate patch, though.

René
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help