Opened 15 years ago
Closed 15 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 by , 15 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 15 years ago
| 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.