[FFmpeg-devel] [PATCH] avcodec/tiff_common: Improve formating of long arrays

Michael Niedermayer michaelni at gmx.at
Thu Oct 3 18:37:27 CEST 2013


On Thu, Oct 03, 2013 at 03:02:42PM +0000, Paul B Mahol wrote:
> On 10/3/13, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Wed, Oct 02, 2013 at 04:35:54PM +0000, Paul B Mahol wrote:
> >> On 9/29/13, Michael Niedermayer <michaelni at gmx.at> wrote:
> >> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> >> > ---
> >> >  libavcodec/tiff_common.c       |   24 +++--
> >> >  tests/ref/fate/exif-image-jpg  |  228
> >> > ++++++++++++++++++++++++++++++++++------
> >> >  tests/ref/fate/exif-image-tiff |    2 +-
> >> >  3 files changed, 208 insertions(+), 46 deletions(-)
> >> >
> >>
> >> Are they really meant to be reformatted?
> >
> > they are arrays of integers
> > printing them as "%d, " lists already reformats them
> 
> I was just curios how would such output be practical.
> Perhaps its meant to stay in binary... but I'm way off-topic now

maybe, i dont know, i guess printing them as ascii number lists is
likely not the intended presentation, still if they are printed that
way it should be done in a readable formatting


> so patch is fine.

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct answer.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131003/35396a68/attachment.asc>


More information about the ffmpeg-devel mailing list