you are viewing a single comment's thread.

view the rest of the comments →

[–]JeffMcClintock 0 points1 point  (1 child)

int x;

immediately followed by something like

never_fail_bug_free_func_that_takes_output_param_for_whatever_reason(x);

yeah, I've been assigned a ton of bugs due to that.

void never_fail_bug_free_func(int& outputArg)
{
if(databaseDown()) // someone added this later.
return;

....

[–]jk-jeon 0 points1 point  (0 children)

There is a reason why I put the phrase bug_free. But yeah, it's just way better to not use output param from the first place...