[PATCH 1/7] Remove value modification decision from plus_minus_two()

[email protected] Wed, 4 Mar 2015 13:09:55 -0800
Newsgroups org.kernel.vger.trinity
Message-ID <[email protected]>
From: Tyson Smith <[email protected]>

---
 interesting-numbers.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/interesting-numbers.c b/interesting-numbers.c
index e77ec80..c0805ab 100644
--- a/interesting-numbers.c
+++ b/interesting-numbers.c
@@ -7,15 +7,14 @@
 static unsigned int plus_minus_two(unsigned int num)
 {
 	/* Now munge it for off-by-ones. */
-	switch (rand() % 5) {
+	switch (rand() % 4) {
 	case 0:	num -= 2;
 		break;
 	case 1:	num -= 1;
 		break;
-	case 2:	return num;
-	case 3:	num += 1;
+	case 2:	num += 1;
 		break;
-	case 4:	num += 2;
+	case 3:	num += 2;
 		break;
 	}
 	return num;
-- 
1.9.1