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:


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 Martin Svensson, 15 years ago

Owner: changed from everyone to Martin Svensson
Status: newassigned

comment:2 by Martin Svensson, 15 years ago

Resolution: fixed
Status: assignedclosed

(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.

Note: See TracTickets for help on using tickets.