On 16.6.2014 17:21, Peter Krempa wrote:
Use consistent formatting of function headers:
- two newlines separating functions
- function return type on separate line
- one argument per line
---
src/util/virjson.c | 287 +++++++++++++++++++++++++++++++++++++++--------------
1 file changed, 215 insertions(+), 72 deletions(-)
diff --git a/src/util/virjson.c b/src/util/virjson.c
index 35a8252..af760a8 100644
--- a/src/util/virjson.c
+++ b/src/util/virjson.c
@@ -63,7 +63,8 @@ struct _virJSONParser {
};
-void virJSONValueFree(virJSONValuePtr value)
+void
+virJSONValueFree(virJSONValuePtr value)
{
size_t i;
if (!value || value->protect)
@@ -97,7 +98,8 @@ void virJSONValueFree(virJSONValuePtr value)
}
-virJSONValuePtr virJSONValueNewString(const char *data)
+virJSONValuePtr
+virJSONValueNewString(const char *data)
{
virJSONValuePtr val;
@@ -116,7 +118,9 @@ virJSONValuePtr virJSONValueNewString(const char *data)
return val;
}
-virJSONValuePtr virJSONValueNewStringLen(const char *data, size_t length)
+
+virJSONValuePtr
+virJSONValueNewStringLen(const char *data, size_t length)
You've missed one function with two arguments.
[..]
ACK with the change above.
Pavel