19 Dec
2008
19 Dec
'08
14:13
>>> On 12/19/2008 at 3:13 PM, Philipp Thomaswrote: > * Dominique Leuenberger (Dominique.Leuenberger@TMF-Group.com) [20081219 > 14:47]: > >> Which ends up to be in this loop: >> >> for(t=1; t m_nInstruments; t++) { // xmms modplug doesn't use slot > zero >> if( (of->Headers[t] = new INSTRUMENTHEADER) == NULL ) return > FALSE; >> memset(of->Headers[t], 0, sizeof(INSTRUMENTHEADER)); >> PATinst(of->Headers[t], t, pat_smptogm(t)); >> } >> > > I'd try > > memset(of->Headers[t], 0, sizeof(of->Headers[t]); > > Philipp Can you explain this? Creating the patch is one thing; but I atually prefer also understanding the background. And I did not specify in my mail that the line failing is the one with PATinst(), just in case that makes a difference. Of course only if you have time for such things... Dominique -- To unsubscribe, e-mail: opensuse-programming+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-programming+help@opensuse.org