This seems trivial, as internal_init() already does the right thing when a return value is used here.

Acked-by: David Hendricks <dhendrix@google.com>

On Wed, May 30, 2012 at 11:48 AM, Niklas Söderlund <niso@kth.se> wrote:
Replace with a return instead, all other plumbing is already in place

Signed-off-by: Niklas Söderlund <niso@kth.se>
---
 internal.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/internal.c b/internal.c
index 5b24577..75e526b 100644
--- a/internal.c
+++ b/internal.c
@@ -298,7 +298,7 @@ int internal_init(void)
                       msg_perr("Proceeding anyway because user forced us to.\n");
               } else {
                       msg_perr("Aborting.\n");
-                       exit(1);
+                       return 1;
               }
       }

--
1.7.10.2


_______________________________________________
flashrom mailing list
flashrom@flashrom.org
http://www.flashrom.org/mailman/listinfo/flashrom



--
David Hendricks (dhendrix)
Systems Software Engineer, Google Inc.