On Wed, Nov 23, 2011 at 11:42 AM, Paul Wise <[email protected]> wrote: > On Wed, Nov 23, 2011 at 12:29 PM, Neutron Soutmun wrote: > >> Considered to not change by now as upstream accepts the manpage patch >> already. >> http://code.google.com/p/flvmeta/source/detail?r=f49070b489892d7f39e84b73cc85293987977623 > > You could send a patch to remove the roff cruft.
OK. I will. >> Maybe, in the next upstream version should includes the manpage and >> the Debian manpage should be dropped. > > Sounds good. :) >> - fprintf (stderr, "JSON: unsupported unicode value: 0x%llX\n", >> (uint64_t)unicode); >> + fprintf (stderr, "JSON: unsupported unicode value: 0x%llX\n", >> (unsigned long long)unicode); >> >> with this patch, compiled without warning. Wondering, I think that >> "uint64_t" = "unsigned long long int". > > Based on a quick web search I think a patch like this based on > inttypes.h and PRIX64 might be better: > > http://sourceware.org/bugzilla/attachment.cgi?id=4393&action=diff I have prepared a patch and forwarded upstream (http://code.google.com/p/flvmeta/issues/detail?id=38#c10) Best regards, Neutron Soutmun -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected] Archive: http://lists.debian.org/CAMQP0s=pdso73wprtl8y6ttfc-wgafodwdoazwk2stsc9wv...@mail.gmail.com
