Opened 13 years ago

Closed 13 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 13 years ago by Martin Svensson

Owner: changed from everyone to Martin Svensson
Status: newassigned

comment:2 Changed 13 years ago by Martin Svensson

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.