On 05/05/2022 05:38, Nick Hanley wrote:
On Wed, May 04, 2022 at 09:49:33AM +0200, Antonio Quartulli wrote:
On 04/05/2022 00:18, Nick Hanley wrote:
On Tue, May 03, 2022 at 02:45:05PM -0400, [email protected] wrote:
Just wondering - why not just do
if (*getstyle(geturi(c)))
setstyle(c, file);
We need to pass the file path returned by getstyle to setstyle.
An alternative would be to keep the original nested invocation and add a
NULL check inside setstyle(). This would avoid adding an extra variable
'file'.
Is there a reason to avoid declaring a variable?
Well, the variable was not there, so the question is: is there a reason
to add it? :-D
IMHO the code is easier without it (as the variable is not used anywhere
else)
Anyway, this is really just my opinion (it was the first time I wrote to
this mailing list), so I am not sure how important it is :)
Regards,
--
Antonio Quartulli
