Github user andrzejdoro commented on a diff in the pull request:

    https://github.com/apache/guacamole-server/pull/205#discussion_r239016330
  
    --- Diff: src/guacd/conf-file.h ---
    @@ -32,10 +37,17 @@
     int guacd_conf_parse_file(guacd_config* conf, int fd);
    --- End diff --
    
    What about hiding `guacd_conf_parse_file()` by making it static?
    Now, `guacd_conf_load` provides this functionality on higher level.


---

Reply via email to