Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use printf %q for simplicity and binary safety #9

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Use printf %q for simplicity and binary safety #9

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
459f98d
Use `printf %q` for simplicity and binary safety
Artoria2e5 Sep 16, 2016
fc00d50
test: binary-safe escape
Artoria2e5 Sep 18, 2016
da8c380
fix deletion of "weird" keys
Artoria2e5 Sep 19, 2016
b296b32
silent printf %q if there is nothing to print
Artoria2e5 Sep 20, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix deletion of "weird" keys 
See thread:
https://lists.gnu.org/archive/html/help-bash/2016-09/msg00020.html

test cases needed.
  • Loading branch information
@Artoria2e5
Artoria2e5 authored Sep 19, 2016
commit da8c3800c6bda1f852f6b36d5172fb3857794f2d
8 changes: 4 additions & 4 deletions quinedb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,11 +97,11 @@ case "$1" in
echo 'OK' >&2
;;
"delete")
unset db["$2"]
unset 'db[$2]'
echo 'OK' >&2
;;
"keys")
for k in "${!db[@]}"; do echo "$(printf %q "$k")"; done >&2
printf '%q\n' "${!db[@]}" >&2
;;
*)
echo "USAGE: quinedb [get k | set k v | delete k | keys]" >&2
Expand Down Expand Up @@ -143,11 +143,11 @@ case "$1" in
echo 'OK' >&2
;;
"delete")
unset db["$2"]
unset 'db[$2]'
echo 'OK' >&2
;;
"keys")
for k in "${!db[@]}"; do echo "$(printf %q "$k")"; done >&2
printf '%q\n' "${!db[@]}" >&2
;;
*)
echo "USAGE: quinedb [get k | set k v | delete k | keys]" >&2
Expand Down