Current FTBFS packages (was Re: [ACTION REQUIRED] Retiring packages for Fedora 21)

Adam Jackson ajax at redhat.com
Wed Jun 18 19:45:19 UTC 2014


On Wed, 2014-06-18 at 15:24 -0400, Przemek Klosowski wrote:
> On 06/18/2014 02:16 PM, Adam Jackson wrote:
> 
> > If I may vent for a moment, I'd like to point out exactly how
> > spurious the blocks usage was (and, implicitly, troll for code
> > review): 
> > http://pkgs.fedoraproject.org/cgit/hfsplus-tools.git/plain/hfsplus-tools-no-blocks.patch 
> > That's right kids, the C89 version is less code even _before_ you
> > count the actual Blocks runtime.
> I should know better than start an argument about programming with
> you, but isn't your patch leaking memory? I don't know how often
> hfsplus tools allocate ctx->preMessage but just overwriting the
> pointer seems off. At least  a comment, maybe?

If bp were a block then yes, but since it's just a function pointer
now...

- ajax



More information about the devel mailing list