Re: [SCM] faad2 packaging branch, master, updated. debian/2.7-4-5-ga421d73

2010-08-18 Thread Fabian Greffrath
Am 17.08.2010 17:40, schrieb Felipe Sateler: No, I don't see anything weird. I do not know the codebase, so I don't know if it is complete, though. Cool, thanks. I'd like to get some feedback from upstream about these patches and then ask again for an upload to Debian. - Fabian ___

Re: [SCM] faad2 packaging branch, master, updated. debian/2.7-4-5-ga421d73

2010-08-17 Thread Felipe Sateler
On 17/08/10 04:44, Fabian Greffrath wrote: > Alright, it's not a secret anymore that I am better at reading C-code > than writing it. ;) Would you please have another look at the patch and > tell me if you see something obviously bogus? No, I don't see anything weird. I do not know the codebase, s

Re: [SCM] faad2 packaging branch, master, updated. debian/2.7-4-5-ga421d73

2010-08-17 Thread Fabian Greffrath
Alright, it's not a secret anymore that I am better at reading C-code than writing it. ;) Would you please have another look at the patch and tell me if you see something obviously bogus? Thanks, - Fabian ___ pkg-multimedia-maintainers mailing list

Re: [SCM] faad2 packaging branch, master, updated. debian/2.7-4-5-ga421d73

2010-08-16 Thread Fabian Greffrath
Am 16.08.2010 18:41, schrieb Adrian Knoth: This seems to be a pretty big patch that should be implemented upstream. As a workaround, one might get along with something like this: #ifndef PATH_MAX #define PATH_MAX 1024 #endif Thanks for the feedback! I wasn't yet finished with this patch, anyw

Re: [SCM] faad2 packaging branch, master, updated. debian/2.7-4-5-ga421d73

2010-08-16 Thread Adrian Knoth
On Mon, Aug 16, 2010 at 12:27:14PM -0400, Felipe Sateler wrote: > > +-char aacFileName[255]; > > +-char audioFileName[255]; > > +-char adtsFileName[255]; > > ++char aacFileName[PATH_MAX + 1]; > > ++char audioFileName[PATH_MAX + 1]; > > ++char adtsFileName[PATH_MAX + 1]; >

Re: [SCM] faad2 packaging branch, master, updated. debian/2.7-4-5-ga421d73

2010-08-16 Thread Felipe Sateler
On 16/08/10 10:36, fabian-gu...@users.alioth.debian.org wrote: > The following commit has been merged in the master branch: > commit a421d733a6b9f24c72326e08bddccdbd5bc0ddf3 > Author: Fabian Greffrath > Date: Mon Aug 16 16:39:32 2010 +0200 > > Extend file name buffers for longer path names.