From cfaf7daa2d42f92e77bac626e5ae755e759ae3f4 Mon Sep 17 00:00:00 2001
From: James Roberts-Thomson <jamesrt@gmail.com>
Date: Tue, 16 Apr 2013 15:53:39 +1200
Subject: Consolidate bash version checks

There were two checks for bash version, the 2nd was superfluous; this
patch removes the duplicate checks, and shifts the bash version check to
being back after the "are we in bash" check.

Change-Id: I57ff8c1fedce80f739c06643d2976d2c1465db1b
---
 envsetup.sh | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/envsetup.sh b/envsetup.sh
index d52d812..2b937f7 100644
--- a/envsetup.sh
+++ b/envsetup.sh
@@ -256,22 +256,18 @@ function settitle()
 
 function addcompletions()
 {
-    if [ "${BASH_VERSINFO[0]}" -lt 4 ] ; then
-        return
-    fi
-
-    local T dir f
-
     # Keep us from trying to run in something that isn't bash.
     if [ -z "${BASH_VERSION}" ]; then
         return
     fi
 
     # Keep us from trying to run in bash that's too old.
-    if [ ${BASH_VERSINFO[0]} -lt 3 ]; then
+    if [ "${BASH_VERSINFO[0]}" -lt 4 ] ; then
         return
     fi
 
+    local T dir f
+
     dirs="sdk/bash_completion vendor/cm/bash_completion"
     for dir in $dirs; do
     if [ -d ${dir} ]; then
-- 
cgit v1.1