[PATCH] remove exit call from internal_init

[Niklas Söderlund]

Replace with a return instead, all other plumbing is already in place

Signed-off-by: Niklas Söderlund 
---
 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;
    	}
    }

[David Hendricks]

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

Acked-by: David Hendricks

On Wed, May 30, 2012 at 11:48 AM, Niklas Söderlund  wrote:
> Replace with a return instead, all other plumbing is already in place
> Signed-off-by: Niklas Söderlund
>
> 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;
>          }
>  }
>
> flashrom mailing list
> [e-mail]
> http://www.flashrom.org/mailman/listinfo/flashrom