Re: [PATCH] %lld support in apr_snprintf()

2010-11-07 Thread Jeff Trawick
On Sun, Nov 7, 2010 at 6:51 PM, Chris Knight wrote: > Exactly, the problem only appears on 64-bit Snow Leopard. See my patch in > Bugzilla, which I've verified. (Unsure if the below would also work, been a > long time since I diagnosed.) What I understood was that %lld is supposed to work, inde

Re: [PATCH] %lld support in apr_snprintf()

2010-11-07 Thread Chris Knight
Exactly, the problem only appears on 64-bit Snow Leopard. See my patch in Bugzilla, which I've verified. (Unsure if the below would also work, been a long time since I diagnosed.) On Nov 7, 2010, at 3:36 PM, Jeff Trawick wrote: > On Sun, Nov 7, 2010 at 5:18 PM, Jeff Trawick wrote: >> On Sun, N

Re: [PATCH] %lld support in apr_snprintf()

2010-11-07 Thread Jeff Trawick
On Sun, Nov 7, 2010 at 5:18 PM, Jeff Trawick wrote: > On Sun, Nov 7, 2010 at 5:07 PM, Sander Temme wrote: >> >> On Nov 4, 2010, at 12:15 PM, William A. Rowe Jr. wrote: >> >>> Looks good here. >> >> If folks find this unproblematic, can someone please commit it?  I don't >> have karma here. >> >>

Re: [PATCH] %lld support in apr_snprintf()

2010-11-07 Thread Jeff Trawick
On Sun, Nov 7, 2010 at 5:07 PM, Sander Temme wrote: > > On Nov 4, 2010, at 12:15 PM, William A. Rowe Jr. wrote: > >> Looks good here. > > If folks find this unproblematic, can someone please commit it?  I don't have > karma here. > > Thanks, looks fine to me; starting to try it out now

Re: [PATCH] %lld support in apr_snprintf()

2010-11-07 Thread Sander Temme
On Nov 4, 2010, at 12:15 PM, William A. Rowe Jr. wrote: > Looks good here. If folks find this unproblematic, can someone please commit it? I don't have karma here. Thanks, S. -- san...@temme.net http://www.temme.net/sander/ PGP FP: FC5A 6FC6 2E25 2DFD 8007 EE23 9BB8 63B0 F51

Bug report for APR [2010/11/07]

2010-11-07 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned