Bug #47512 for Bloom-Faster: suggestions.c printf

Thu Jul 02 11:42:44 2009 jtk [...] northwestern.edu - Ticket created

Subject:

suggestions.c printf

Thanks for addressing the report 46455 so quickly! However, I found at least one minor issue still outstanding from that report. In suggestions.c there is: printf("m is %ld (from %ld),k is %d\n",(long)stat->elements, (long)stat->capacity,stat->ideal_hashes); That gets send to stdout, which is less than ideal, especially when used in automated tools that might expecting silence. John

Thu Jul 02 14:12:32 2009 dvryaboy [...] cpan.org - Correspondence added

Hey John, Sorry, that's my fault -- missed it. I've fixed it in our development version, we'll push a new minor release soon with some other stuff. Thanks for bringing it to our attention. -D On Thu Jul 02 11:42:44 2009, JTK wrote: Show quoted text

> Thanks for addressing the report 46455 so quickly! However, I found at > least one minor issue still outstanding from that report. In > suggestions.c there is: > > printf("m is %ld (from %ld),k is %d\n",(long)stat->elements, > (long)stat->capacity,stat->ideal_hashes); > > That gets send to stdout, which is less than ideal, especially when used > in automated tools that might expecting silence. > > John

Thu Jul 02 14:12:34 2009 The RT System itself - Status changed from 'new' to 'open'

Thu Dec 03 18:32:07 2009 jtk [...] northwestern.edu - Correspondence added

Bump. Not critical, but would love to see that updated release with this resolved when you get a chance. I can't imagine you need a patch for this, but if you do I'll be happy to send it in. John

Tue Dec 15 14:08:18 2009 dvryaboy [...] cpan.org - Correspondence added

Right, sorry. No, I don't need a patch :-). Will package it up today/tomorrow. On Thu Dec 03 18:32:07 2009, JTK wrote: Show quoted text

> Bump. Not critical, but would love to see that updated release with > this resolved when you get a chance. I can't imagine you need a patch > for this, but if you do I'll be happy to send it in. > > John

Wed Dec 16 14:31:22 2009 dvryaboy [...] cpan.org - Correspondence added

There is a new version in rev 7 (head of trunk, as of this writing) of our svn repo for Bloom::Faster here: http://sourceforge.net/projects/bloom-faster/develop Seems a bit silly to release a new version with just this change, so we'll come up with something meatier for 1.6.1. But you can fetch it from svn if you like.

Sat Jun 12 20:17:12 2010 palvaro [...] cpan.org - Taken

Sat Jun 12 20:17:31 2010 palvaro [...] cpan.org - Correspondence added

fixed in v1.7

Sat Jun 12 20:17:32 2010 palvaro [...] cpan.org - Status changed from 'open' to 'resolved'