aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTeddy Wing2016-07-24 17:37:12 -0400
committerTeddy Wing2016-07-24 17:37:12 -0400
commit7e02cf216c6e0d1592a4f8ac1a3827a01a8b37ed (patch)
tree7d49b364775a4019b90a156dde8b576e9e38a09b
parent3866bef812e5967f1b24c42239cb6436ae773502 (diff)
downloadqcd-7e02cf216c6e0d1592a4f8ac1a3827a01a8b37ed.tar.bz2
qcd: Prefix helper functions with `__qcd_`
Wasn't thinking about the fact that these helper functions get exposed to the global namespace, which is _such_ a bad idea. Rename these functions to something specific to the program to make them harder to randomly come across and to have a better chance of not overriding something that already exists.
-rwxr-xr-xqcd42
1 files changed, 21 insertions, 21 deletions
diff --git a/qcd b/qcd
index ea51670..3d966bb 100755
--- a/qcd
+++ b/qcd
@@ -10,7 +10,7 @@ QCD_CONFIG_DIR=${QCD_CONFIG_DIR:-"${XDG_CONFIG_HOME}/qcd"}
QCD_DATABASE_FILE=${QCD_DATABASE_FILE:-"${QCD_CONFIG_DIR}/database"}
-function print_usage () {
+function __qcd_print_usage () {
cat 1>&2 <<__EOF__
Usage:
qcd SHORTCUT
@@ -26,55 +26,55 @@ Usage:
__EOF__
}
-function setup () {
+function __qcd_setup () {
mkdir -p $QCD_CONFIG_DIR
touch $QCD_DATABASE_FILE
}
-function absolute_path () {
+function __qcd_absolute_path () {
(cd "$(dirname "$1")" && printf "%s/%s\n" "$(pwd)" "$(basename "$1")")
}
-function shortcut_exists () {
+function __qcd_shortcut_exists () {
local shortcut=$1
return $(awk '{ print $1 }' $QCD_DATABASE_FILE | grep "^${shortcut}$" >/dev/null 2>&1; echo $?)
}
-function add_shortcut () {
+function __qcd_add_shortcut () {
local shortcut=$1
local path=$2
# Don't add the shortcut if it already exists
- if ! shortcut_exists $shortcut; then
- path=$(absolute_path "$path")
+ if ! __qcd_shortcut_exists $shortcut; then
+ path=$(__qcd_absolute_path "$path")
echo "${shortcut} ${path}" >> $QCD_DATABASE_FILE
fi
}
-function change_shortcut () {
+function __qcd_change_shortcut () {
local shortcut=$1
local path=$2
- if shortcut_exists $shortcut; then
- path=$(absolute_path "$path")
+ if __qcd_shortcut_exists $shortcut; then
+ path=$(__qcd_absolute_path "$path")
# The `//\//\/` escapes slashes in the path so that `sed` doesn't complain
sed -i.bak -E "s/^${shortcut} .+$/${shortcut} ${path//\//\/}/" $QCD_DATABASE_FILE
fi
}
-function remove_shortcut () {
+function __qcd_remove_shortcut () {
local shortcut=$1
- if shortcut_exists $shortcut; then
+ if __qcd_shortcut_exists $shortcut; then
sed -i.bak -E "/^${shortcut} .+/d" $QCD_DATABASE_FILE
fi
}
-function change_directory () {
+function __qcd_change_directory () {
local shortcut=$1
- if shortcut_exists $shortcut; then
+ if __qcd_shortcut_exists $shortcut; then
local directory=$(awk -v shortcut="$shortcut" '{
if ($1 == shortcut)
{ sub(/[[:space:]]*([^[:space:]]+[[:space:]]+)/,""); print }
@@ -86,11 +86,11 @@ function change_directory () {
function qcd () {
if [[ $# < 1 ]]; then
- print_usage
+ __qcd_print_usage
return
fi
- setup
+ __qcd_setup
local command=$1
local shortcut=$2
@@ -98,19 +98,19 @@ function qcd () {
case $command in
-a)
- add_shortcut $shortcut "$path"
+ __qcd_add_shortcut $shortcut "$path"
;;
-c)
- change_shortcut $shortcut "$path"
+ __qcd_change_shortcut $shortcut "$path"
;;
-r)
- remove_shortcut $shortcut
+ __qcd_remove_shortcut $shortcut
;;
-h|--help)
- print_usage
+ __qcd_print_usage
;;
*)
- change_directory $command
+ __qcd_change_directory $command
;;
esac
}