Opened 14 years ago
Closed 14 years ago
#1328 closed enhancement (fixed)
Remove all or most 'continue' statements from the code
| Reported by: | Nicklas Nordborg | Owned by: | Martin Svensson |
|---|---|---|---|
| Priority: | critical | Milestone: | BASE 2.13 |
| Component: | core | Version: | |
| Keywords: | Cc: |
Description
The 'continue' statement is dangerous. It is very easy to miss that this jumps to an unexpected place in the code. See for example #1327. As far as I know we are not using 'continue' very much, but I think that we should really try to replace it with something better in as many places as we can. As part of this ticket the coding guidelines should also be updated with this information.
Change History (2)
comment:1 Changed 14 years ago by
| Owner: | changed from everyone to Martin Svensson |
|---|---|
| Status: | new → assigned |
comment:2 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
(In [5014]) Fixes #1328 Remove all or most 'continue' statements from the code. The finally conclusion was to not remove any continue:s and instead mark them with a clear comment. The coding-guidelines have also been updated with info how to continue statements should be used.