On 01/27/2012 12:17 PM, Aneesh Dogra wrote:

> ---
>  libavcodec/sunrast.c |   13 +++++++++++++
>  1 files changed, 13 insertions(+), 0 deletions(-)
> 
> diff --git a/libavcodec/sunrast.c b/libavcodec/sunrast.c
> index 6da57d1..187eed6 100644
> --- a/libavcodec/sunrast.c
> +++ b/libavcodec/sunrast.c
> @@ -23,12 +23,25 @@
>  #include "libavutil/imgutils.h"
>  #include "avcodec.h"
>  
> +/* Both Old and Standard formats are the same.
> + * They indicate that the image data within the file is not compressed */
>  #define RT_OLD          0
>  #define RT_STANDARD     1


Which one indicates that the data is not compressed?

> +
> +/* The Byte-encoded type indicates that the image data is compressed using a
> + * Run-length encoding scheme */
>  #define RT_BYTE_ENCODED 2


Do not capitalize "Run-length".
And capitalize both parts of a hyphenated word like "Byte-Encoded"

> +
>  #define RT_FORMAT_RGB   3
> +
> +/* The TIFF and IFF format types indicate that the raster file was originally
> + * converted from either of these file formats. */
>  #define RT_FORMAT_TIFF  4
>  #define RT_FORMAT_IFF   5


Mention something about not knowing the details of an image with these
types and not having any samples (if this is true).

Thanks,
Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to