X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=CodingStyle;h=77fa90c733be38f090d937d05950ffc47ef29122;hp=6e7adfc4e9dcd7d9a5a1397d41d969d66e970752;hb=1f2f41593be38847883eaf11ca8f5db08adec5a1;hpb=21073eaa900a731710832c1897d6200637d7ddb0 diff --git a/CodingStyle b/CodingStyle index 6e7adfc4e..77fa90c73 100644 --- a/CodingStyle +++ b/CodingStyle @@ -14,11 +14,32 @@ single-line if/else statements. Please refer to: - Linux kernel scripts/checkpatch.pl for a script which verify the patch coding style. +Error handling: +------------- + +We ask to use one single return point in a function. For that, we uses the +"goto" statement for the error handling creating one single point for error +handling and return code. See the following example: + +int some_function(...) +{ + int ret; + [...] + + if (ret != 0) { + goto error; + } + + [...] +error: + return ret; +} + Commenting: ------------- Every function MUST have a comment above it even if the function is trivial. -Please add non-trivial comments/documentation as much you can in the code. Poor -comments WILL be rejected upon merging so please pay attention to this details -because we do! +Please add non-trivial comments/documentation as much as you can in the code. +Poor comments WILL be rejected upon merging so please pay attention to this +details because we do!