From adfd3c10985bffeebd43a710b3f00f836f66569b Mon Sep 17 00:00:00 2001
From: Kevin Ballard <kevin@sb.org>
Date: Thu, 21 Jun 2012 19:31:25 -0700
Subject: [PATCH] Don't blow away $status in switch statements

Tweak the switch and case builtins to not blow away $status
inappropriately. Fixes issue #161.
---
 builtin.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/builtin.cpp b/builtin.cpp
index 219363b3c..becff9324 100644
--- a/builtin.cpp
+++ b/builtin.cpp
@@ -3540,6 +3540,7 @@ static int builtin_switch( parser_t &parser, wchar_t **argv )
 		parser.current_block->state1<wcstring>() = argv[1];
 		parser.current_block->skip=1;
 		parser.current_block->state2<int>() = 0;
+        res = proc_get_last_status();
 	}
 	
 	return res;
@@ -3568,7 +3569,7 @@ static int builtin_case( parser_t &parser, wchar_t **argv )
 	
 	if( parser.current_block->state2<int>() )
 	{
-		return STATUS_BUILTIN_OK;
+		return proc_get_last_status();
 	}
 	
 	for( i=1; i<argc; i++ )
@@ -3588,7 +3589,7 @@ static int builtin_case( parser_t &parser, wchar_t **argv )
 		}
 	}
 	
-	return STATUS_BUILTIN_OK;
+	return proc_get_last_status();
 }