[GitHub] [guacamole-server] jmuehlner commented on a change in pull request #333: GUACAMOLE-1302: Add support for forcing lossless compression on VNC/RDP connections.

[GitBox <gi...@apache.org>]

jmuehlner commented on a change in pull request #333:
URL: https://github.com/apache/guacamole-server/pull/333#discussion_r587037175



##########
File path: src/common/common/display.h
##########
@@ -228,5 +235,27 @@ void guac_common_display_free_layer(guac_common_display* display,
 void guac_common_display_free_buffer(guac_common_display* display,
         guac_common_display_layer* display_buffer);
 
+/**
+ * Sets the overall lossless compression policy of the given display to the
+ * given value, affecting all current and future layers/buffers maintained by
+ * the display. By default, newly-created displays will use lossy compression
+ * for graphical updates when heuristics determine that doing so is
+ * appropriate. Specifying a non-zero value here will force all graphical
+ * updates to always use lossless compression, whereas specifying zero will
+ * restore the default policy.
+ *
+ * Note that this can also be adjusted on a per-layer / per-buffer basis with
+ * guac_common_surface_set_lossless().
+ *
+ * @param display
+ *     The display to modify

Review comment:
       Looks like there's a period missing here on this param doc - same as `guac_common_surface_set_lossless`.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org