do not percent-encode: ',' or '-' or '.' it looks ugly
authorHiltjo Posthuma <hiltjo@codemadness.org>
Tue, 14 Dec 2021 19:51:02 +0000 (20:51 +0100)
committerHiltjo Posthuma <hiltjo@codemadness.org>
Tue, 14 Dec 2021 19:52:18 +0000 (20:52 +0100)
stagit-index.c
stagit.c

index 8a53463d047cc00251f788f93be3fefa382ddb16..087ae1e95023430256c6adce8315e68bdd740ff9 100644 (file)
@@ -38,8 +38,8 @@ percentencode(FILE *fp, const char *s, size_t len)
 
        for (i = 0; *s && i < len; s++, i++) {
                uc = *s;
-               /* NOTE: do not encode '/' for paths */
-               if (uc < '/' || uc >= 127 || (uc >= ':' && uc <= '@') ||
+               /* NOTE: do not encode '/' for paths or ",-." */
+               if (uc < ',' || uc >= 127 || (uc >= ':' && uc <= '@') ||
                    uc == '[' || uc == ']') {
                        putc('%', fp);
                        putc(tab[(uc >> 4) & 0x0f], fp);
index 6e0c15e03abaeaefb42d48aebc86aa25bb71e8b9..22fc2aa3c2d71d0825675afffa7408691d2e6a82 100644 (file)
--- a/stagit.c
+++ b/stagit.c
@@ -369,8 +369,8 @@ percentencode(FILE *fp, const char *s, size_t len)
 
        for (i = 0; *s && i < len; s++, i++) {
                uc = *s;
-               /* NOTE: do not encode '/' for paths */
-               if (uc < '/' || uc >= 127 || (uc >= ':' && uc <= '@') ||
+               /* NOTE: do not encode '/' for paths or ",-." */
+               if (uc < ',' || uc >= 127 || (uc >= ':' && uc <= '@') ||
                    uc == '[' || uc == ']') {
                        putc('%', fp);
                        putc(tab[(uc >> 4) & 0x0f], fp);