Github user mike-jumper commented on a diff in the pull request:

    https://github.com/apache/guacamole-server/pull/159#discussion_r173688157
  
    --- Diff: src/guacenc/ffmpeg-compat.c ---
    @@ -165,3 +197,62 @@ int guacenc_avcodec_encode_video(guacenc_video* video, 
AVFrame* frame) {
     #endif
     }
     
    +AVCodecContext* guacenc_build_avcodeccontext(AVStream* stream,
    +        AVCodec* codec,
    --- End diff --
    
    Please follow the general style of the surrounding code. This function 
stands out as the only one to dedicate a separate line to each parameter, 
regardless of line length.


---

Reply via email to