Josef Grosch via Dng said on Thu, 29 Jul 2021 15:55:03 -0700
>On 7/28/21 1:12 PM, Steve Litt wrote:
>> [ Beverity ]
>> Does anyone have other list items to add?
>>
>
>Now that you got me thinking;
>
>One should be explicit instead of implicit. I see this in code all the
>time and it drives me crazy
>
>
>// NO
>
>if (condition)
>
> doSomething();
>
>
>// YES
>
>if (condition) {
>
> doSomething();
>
>}
The preceding is reasonable. But sometimes, readability is best served
with:
if (condition) doSomething();
I think the preceding should be allowed, when it makes things clearer.
if (no_record_found) show_error_and_exit("No record found", 1);
SteveT
Steve Litt
Spring 2021 featured book: Troubleshooting Techniques of the Successful
Technologist http://www.troubleshooters.com/techniques
_______________________________________________
Dng mailing list
Dng@lists.dyne.org
https://mailinglists.dyne.org/cgi-bin/mailman/listinfo/dng
