On 24 Sep, Gopu Govindaswamy wrote :
> @@ -173,7 +175,7 @@
>   \param   accessUnitsOut      output bitstream
>   \retval                      number of encoded pictures
>   */
> -int TEncTop::encode(bool flush, const x265_picture_t* pic_in, x265_picture_t 
> *pic_out, AccessUnit& accessUnitOut)
> +int TEncTop::encode(bool flush, const x265_picture_t* pic_in, x265_picture_t 
> *pic_out, NALUnitEBSP **nalunits)


Please update the doxygen above the function.

> -    for (AccessUnit::const_iterator it = accessUnit.begin(); it != 
> accessUnit.end(); it++)
> +    int count = 0;
> +    for (;nalunits[count] != NULL; count++)

You should be able to define the count in the for first parameter, if I
am not mistaken.

The rest, LGTM.

Best regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

Reply via email to