mike-jumper commented on a change in pull request #159:
URL: https://github.com/apache/guacamole-server/pull/159#discussion_r440672609
##########
File path: src/guacenc/ffmpeg-compat.h
##########
@@ -78,5 +88,81 @@
*/
int guacenc_avcodec_encode_video(guacenc_video* video, AVFrame* frame);
+/**
+ * Creates and sets up the AVCodecContext for the appropriate version
+ * of libavformat installed. The AVCodecContext will be built, but
+ * the AVStream will also be affected by having its time_base field
+ * set to the value passed into this function.
+ *
+ * @param stream
+ * The open AVStream
Review comment:
As with JavaDoc, the formatting of these Doxygen comments requires that
the description of parameters, return values, etc. all end with periods. This
is correct for the other new code - some sneaky creature just went through this
one and ate all the periods. ;)
----------------------------------------------------------------
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:
[email protected]