FGCOM/jansson
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use sizeof(seq) instead of magic number 13

Browse Source 
Be more future proof by keeping the number of magic constants low.
This commit is contained in:
Jonas Jonsson 2015-10-05 13:59:09 +02:00
parent 02464862ed
commit f7331c7194
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/dump.c
View File

@ -134,7 +134,7 @@ static int dump_string(const char *str, size_t len, json_dump_callback_t dump, v
/* codepoint is in BMP */ /* codepoint is in BMP */
if(codepoint < 0x10000) if(codepoint < 0x10000)
{ {
snprintf(seq, 13, "\\u%04X", codepoint); snprintf(seq, sizeof(seq), "\\u%04X", codepoint);
length = 6; length = 6;
} }
@ -147,7 +147,7 @@ static int dump_string(const char *str, size_t len, json_dump_callback_t dump, v
first = 0xD800 | ((codepoint & 0xffc00) >> 10); first = 0xD800 | ((codepoint & 0xffc00) >> 10);
last = 0xDC00 | (codepoint & 0x003ff); last = 0xDC00 | (codepoint & 0x003ff);
snprintf(seq, 13, "\\u%04X\\u%04X", first, last); snprintf(seq, sizeof(seq), "\\u%04X\\u%04X", first, last);
length = 12; length = 12;
} }