blender
/
blender
136
406
Fork 589
Code Issues 6.2k Pull Requests 757 Projects 19 Wiki Activity

BLI_string: support escaping additional control character 

Add support for escaping \a, \b & \f for completeness,
currently it's not required.
This commit is contained in:
Campbell Barton 2020-12-10 13:45:57 +11:00
parent 15d801625c
commit 82e1b65d91
2 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
source/blender/blenlib/intern/string.c
View File

@ -340,7 +340,10 @@ size_t BLI_str_escape(char *__restrict dst, const char *__restrict src, const si
if (ELEM(c, '\\', '"') || /* Use as-is. */
((c == '\t') && ((void)(c = 't'), true)) || /* Tab. */
((c == '\n') && ((void)(c = 'n'), true)) || /* Newline. */
((c == '\r') && ((void)(c = 'r'), true))) /* Carriage return. */
((c == '\r') && ((void)(c = 'r'), true)) || /* Carriage return. */
((c == '\a') && ((void)(c = 'a'), true)) || /* Bell. */
((c == '\b') && ((void)(c = 'b'), true)) || /* Backspace. */
((c == '\f') && ((void)(c = 'f'), true))) /* Form-feed. */
{
if (UNLIKELY(len + 1 >= dst_maxncpy)) {
/* Not enough space to escape. */
@ -378,7 +381,11 @@ size_t BLI_str_unescape(char *__restrict dst, const char *__restrict src, const
if (((c_next == '"') && ((void)(c = '"'), true)) || /* Quote. */
((c_next == '\\') && ((void)(c = '\\'), true)) || /* Backslash. */
((c_next == 't') && ((void)(c = '\t'), true)) || /* Tab. */
((c_next == 'n') && ((void)(c = '\n'), true))) /* Newline. */
((c_next == 'n') && ((void)(c = '\n'), true)) || /* Newline. */
((c_next == 'r') && ((void)(c = '\r'), true)) || /* Carriage return. */
((c_next == 'a') && ((void)(c = '\a'), true)) || /* Bell. */
((c_next == 'b') && ((void)(c = '\b'), true)) || /* Backspace. */
((c_next == 'f') && ((void)(c = '\f'), true))) /* Form-feed. */
{
i++;
src++;

9
source/blender/blenlib/tests/BLI_string_test.cc
View File

@ -863,11 +863,14 @@ TEST_F(StringEscape, Control)
{"\n", "\\n"},
{"\r", "\\r"},
{"\t", "\\t"},
{"\a", "\\a"},
{"\b", "\\b"},
{"\f", "\\f"},
{"A\n", "A\\n"},
{"\nA", "\\nA"},
{"\n\r\t", "\\n\\r\\t"},
{"\n_\r_\t", "\\n_\\r_\\t"},
{"\n\\\r\\\t", "\\n\\\\\\r\\\\\\t"},
{"\n\r\t\a\b\f", "\\n\\r\\t\\a\\b\\f"},
{"\n_\r_\t_\a_\b_\f", "\\n_\\r_\\t_\\a_\\b_\\f"},
{"\n\\\r\\\t\\\a\\\b\\\f", "\\n\\\\\\r\\\\\\t\\\\\\a\\\\\\b\\\\\\f"},
};
testEscapeWords(escaped);