Opened 13 years ago

Closed 13 years ago

#7352 closed enhancement (invalid)

Removing goto from boot loader main.cpp

Reported by: Finder Owned by: axeld
Priority: normal Milestone: R1
Component: System/Boot Loader Version: R1/Development
Keywords: goto Cc:
Blocked By: Blocking:
Platform: All

Description

This is just a little patch to remove a goto from the main.cpp of the boot loader. I made this because:

  1. coding guidelines
  2. goto is evil
  3. ​http://xkcd.com/292/

Attachments (1)

loadermain-nogoto.patch​ (800 bytes ) - added by Finder 13 years ago.

Download all attachments as: .zip

Change History (4)

by Finder, 13 years ago

Attachment: loadermain-nogoto.patch​ added

comment:1 by Finder, 13 years ago

patch: 0 → 1

comment:2 by korli, 13 years ago

Finder, you must be kidding, right?

comment:3 by axeld, 13 years ago

Resolution: → invalid
Status: new → closed

While goto should be avoided generally, using goto to bail out like this, and do proper cleanup is perfectly fine IMO, and it's actually easier to follow than your code. Thanks for your patch, anyway, though!

Just look at src/system/kernel/fs/vfs.cpp if you want to shudder a bit more :-)

Note: See TracTickets for help on using tickets.