Github user mike-jumper commented on a diff in the pull request:
https://github.com/apache/incubator-guacamole-server/pull/120#discussion_r150634771
--- Diff: src/common/display.c ---
@@ -99,6 +99,19 @@ static void
guac_common_display_free_layers(guac_common_display_layer* layers,
}
+/**
+ * Allocates a display and a cursor which are used to represent the remote
+ * display and cursor. If the allocation fails then the function returns
NULL.
--- End diff --
Same here - much better, but return values should also be documented with
`@return`.
---
