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 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,

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

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, so

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 fab...@greffrath.com Date: Mon Aug 16 16:39:32 2010 +0200 Extend file name buffers for

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]; I believe