[Asterisk-code-review] json: Add UTF-8 check call. (asterisk[13])

[George Joseph]

George Joseph has posted comments on this change.

Change subject: json: Add UTF-8 check call.
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/4095/1/include/asterisk/json.h
File include/asterisk/json.h:

Line 240: int ast_json_utf8_check(const char *str);
This gets called a lot as ast_json_utf8_check(str) ? str : ""
How about adding a convenience macro that does that for you?


-- 
To view, visit https://gerrit.asterisk.org/4095
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I36d3d750b6f5f1a110bc69ea92b435ecdeeb2a99
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-HasComments: Yes