Mailinglist Archive: opensuse-bugs (8956 mails)

< Previous Next >
[Bug 214105] supertux-0.1.3-28: mising return statement
  • From: bugzilla_noreply@xxxxxxxxxx
  • Date: Wed, 25 Oct 2006 05:01:26 -0600 (MDT)
  • Message-id: <20061025110126.9A12CEBC@xxxxxxxxxxxxxxxxxxxxxx>

dcb314@xxxxxxxxxxx changed:

What |Removed |Added
CC| |dcb314@xxxxxxxxxxx
Resolution|INVALID |

------- Comment #4 from dcb314@xxxxxxxxxxx 2006-10-25 05:01 MST -------
(In reply to comment #3)
> assert already does this job.

I think we all know what assert does.

I tried to suggest that there are better ways
of dealing with the problem than crashing.

> Adding return afterwards is a cosmetic change.

As I tried to indicate above, no it isn't.

Putting return statements at the end of non-void functions
[ even in code that isn't supposed to be executed in normal
operation] is a good thing to do.

I strongly recommend re-reading what I wrote.

Configure bugmail:
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

< Previous Next >