Re: RFS: flvmeta (updated package)
On Wed, Nov 23, 2011 at 11:42 AM, Paul Wise <pabs@debian.org> 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
Reply to: