From 9f75e490ed4f6bd614af3f068547a7577a2d9eaa Mon Sep 17 00:00:00 2001
From: lace <>
Date: Fri, 14 Jul 2006 14:32:09 +0000
Subject: [PATCH] Optimized _bash_profile_addpath() performance.

---
 .bashrc | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/.bashrc b/.bashrc
index 1459432..29d098f 100644
--- a/.bashrc
+++ b/.bashrc
@@ -30,11 +30,12 @@ fi
 
 if [ -f /etc/bashrc ];then source /etc/bashrc;fi
 
+_bash_profile_addpath_fini=""
 function _bash_profile_addpath	# <variablename>,<dirpathname>
 {
 	local _varname _varval _varvalold _dirpathname _opt_f _opt_s
 	_opt_f=0;_opt_s=0
-	unset OPTIND
+	unset -v OPTIND
 	while getopts fs got;do
 		case "$got" in
 			[fs]) eval "_opt_$got=1" ;;
@@ -47,17 +48,21 @@ function _bash_profile_addpath	# <variablename>,<dirpathname>
 	if [ $_opt_s = 1 -o -z "`eval "echo \\${$_varname}"`" ];then
 		eval "$_varname='$_dirpathname'"
 	else
+		# We want to prevent duplicates due to .bashrc reentrancy across fork(2)s. It is array!
+		_bash_profile_addpath_fini="$_bash_profile_addpath_fini $_varname"
 		# We want to override any original directories, place dirpathname first!
-		# We want to prevent duplicates due to .bashrc reentrancy across fork(2)s.
-		eval "_varval=\":\${$_varname}:\""
-		until [ "$_varval" = "$_varvalold" ];do
-			_varvalold="$_varval"
-			eval "_varval=\"$(echo "$_varval"|sed "s#:$_dirpathname:#:#g")\""
-		done
-		eval "$_varname=\"$_dirpathname$(echo "$_varval"|sed "s#:\$##g")\""
+		eval "$_varname=\"$_dirpathname:\${$_varname}\""
 	fi
 	export "$_varname"
 }
+function _bash_profile_addpath_fini
+{
+	local _varname
+	for _varname in `echo "$_bash_profile_addpath_fini"|tr ' ' '\n'|sort -u`;do
+		eval "$_varname=\"`eval $(echo "echo \"\\$$_varname\"")|awk -v RS=':' '/[^\n]/{if (!x[$0]++) e=e":"$0;}END{print substr(e,2)}'`\""
+		done
+	unset -v _bash_profile_addpath_fini
+}
 
 # Use reverse order of preferred directories:
 # gid(1) dependency on: /opt/sfw/bin
@@ -108,6 +113,8 @@ for _perldir in $(eval "echo {/usr,$HOME}{/lib{,64}/perl{,5},/share/perl}{,/site
 unset _perldir
 
 unset -f _bash_profile_addpath
+_bash_profile_addpath_fini
+unset -f _bash_profile_addpath_fini
 
 for i in tar ls find cp mv ln md5sum du grep ar chown chmod tail;do
 	# Prevent g$i from system directory overriding customized $i binary
-- 
1.8.3.1