Re: [PATCH] libdvbv5: more fixes in the T2 delivery descriptor handler

2013-12-19 Thread Nikolaus Schulz
On Wed, Dec 18, 2013 at 10:49:31AM -0200, Mauro Carvalho Chehab wrote: > > * Properly use lengths of centre_frequency loop and subcell_info loop > > (they count bytes, not entries) > > I'm not sure about this one. On all other similar descriptors, we're storing > the number of elements at the de

Re: [PATCH] libdvbv5: more fixes in the T2 delivery descriptor handler

2013-12-19 Thread Mauro Carvalho Chehab
Em Thu, 19 Dec 2013 03:37:48 +0100 Nikolaus Schulz escreveu: > Hi Mauro, > > I didn't have the time yet to look at your comments and that patch > again, but: > > Please note that my email address is shut down as > of today[1]. > > Please use my alternate address, , for any further replies > o

Re: [PATCH] libdvbv5: more fixes in the T2 delivery descriptor handler

2013-12-18 Thread Nikolaus Schulz
Hi Mauro, I didn't have the time yet to look at your comments and that patch again, but: Please note that my email address is shut down as of today[1]. Please use my alternate address, , for any further replies or followups. Maybe Patchwork[2] should also be updated accordingly? Thanks, Nikol

Re: [PATCH] libdvbv5: more fixes in the T2 delivery descriptor handler

2013-12-18 Thread Mauro Carvalho Chehab
Thanks for the patch! Em Thu, 5 Dec 2013 16:10:03 +0100 Nikolaus Schulz escreveu: > * Fix a couple of memcpy calls, and remove a bogus one That changes look OK to me. > * Properly use lengths of centre_frequency loop and subcell_info loop > (they count bytes, not entries) I'm not sure about

[PATCH] libdvbv5: more fixes in the T2 delivery descriptor handler

2013-12-05 Thread Nikolaus Schulz
* Fix a couple of memcpy calls, and remove a bogus one * Properly use lengths of centre_frequency loop and subcell_info loop (they count bytes, not entries) Signed-off-by: Nikolaus Schulz --- lib/libdvbv5/descriptors/desc_t2_delivery.c | 35 ++- 1 files changed, 18 in