Argument names different between declaration and definition

[rolegic]

Style warnings raised by cppcheck

cJSON.c:224: style: Function 'cJSON_Delete' argument 1 names different: declaration 'c' definition 'item'.
cJSON.c:1271: style: Function 'cJSON_PrintPreallocated' argument 2 names different: declaration 'buffer' definition 'buf'.
cJSON.c:1271: style: Function 'cJSON_PrintPreallocated' argument 3 names different: declaration 'length' definition 'len'.
cJSON.c:1272: style: Function 'cJSON_PrintPreallocated' argument 4 names different: declaration 'format' definition 'fmt'.
cJSON.c:2460: style: Function 'cJSON_CreateBool' argument 1 names different: declaration 'boolean' definition 'b'.

• cJSON_Delete

  CJSON_PUBLIC(void) cJSON_Delete(cJSON *c);
  CJSON_PUBLIC(void) cJSON_Delete(cJSON *item)
  

• PrintPreallocated

  cJSON_PrintPreallocated(cJSON *item, char *buffer, const int length, const cJSON_bool format);
  cJSON_PrintPreallocated(cJSON *item, char *buf, const int len, const cJSON_bool fmt)
  

• cJSON_CreateBool

  cJSON_CreateBool(cJSON_bool boolean)
  cJSON_CreateBool(cJSON_bool b)
  

[FSMaxB]

Thanks. I fixed these on the develop branch.

I might add cppcheck to the continuous integration so that it gets checked automatically in the future.

[FSMaxB]

Or not, for now there are too many false positives.