[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: flvmeta (updated package)



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.

> 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

-- 
bye,
pabs

http://wiki.debian.org/PaulWise


Reply to: