<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>Patch dropped. It has been included in a more comprehensive patch where 
return values are added when conversion to void is not possible.<br><br>Nicola<br><br><div>> Date: Mon, 2 Mar 2015 15:34:54 -0500<br>> From: feranick@hotmail.com<br>> To: cinelerra@lists.cinelerra-cv.org<br>> Subject: Re: [CinCV TNG] [PATCH] Fix missing return in handle_event in zoombar.C<br>> <br>> <br>> <br>> On 3/2/15 3:00 PM, Johannes Sixt wrote:<br>> > Am 02.03.2015 um 20:43 schrieb Nicola Ferralis:<br>> >> Hi Hannes,<br>> >><br>> >> Thanks. Early on I went on a file by file patch spree, which included<br>> >> most of the recent patches. However, I moved to a more inclusive patch<br>> >> (like the one I just submitted yesterday for the int to void), which<br>> >> captures (hopefully) most cases found by coverity. I plan to do the same<br>> >> going forward.<br>> ><br>> > That patch I like! (And it looks reasonable from a cursory read.)<br>> <br>> Yes. BTW, I am adding even more to that patch, originally I was trying <br>> to keep the size of the patch from being too large. It currently only <br>> covers /cinelerra and guicast/ but based on this suggestion, I will <br>> include also files from /plugins.<br>> <br>> ><br>> >> As for the clear_handle, that sort of slipped through. Since I have not<br>> >> pushed yet any of this individual patches, I'd be happy to repackage<br>> >> them by their fix strategy (i.e. adding returns, removing functions,<br>> >> changing to voids etc).<br>> ><br>> > Great! But please complete your adventure before you push out any patchs.<br>> <br>> OK, sounds good.<br>> <br>> ><br>> > Please do not submit an updated version of a large patch, such as your<br>> > earlier int-to-void patch, because sifting through what was already seen<br>> > and what is new would mean a lot of work for the reviewers. Feel free to<br>> > submit incremental patches for review, but at the end please collect<br>> > those with the same topic into a single patch.<br>> <br>> OK, will do.<br>> <br>> ><br>> > Thanks,<br>> > -- Hannes<br>> ><br>> >><br>> >> Thanks,<br>> >> Nicola<br>> >><br>> >> On 3/2/15 2:36 PM, Johannes Sixt wrote:<br>> >>> Am 02.03.2015 um 00:24 schrieb Nicola Ferralis:<br>> >>>> Throughout cinelerra, handle_event() return 1. Some calls miss<br>> >>>> the return, such as AutoTypeMenu::handle_event() in zoombar.C<br>> >>>> This patch adds return 1 to this call.<br>> >>><br>> >>> You seem to discover even more handle_event() implementations that need<br>> >>> treatment as you dig through coverity reports.<br>> >>><br>> >>> Would you kindly collect all of them into a single patch instead of<br>> >>> letting them trickle in one by one?<br>> >>><br>> >>> Concerning the other kinds of fixes (adding returns, removing entire<br>> >>> functions, etc.) let me point out that your work is very much<br>> >>> appreciated. But please finish your exploration, and then suggest a<br>> >>> reasonable separation of patches into a few topics. Having dozens of<br>> >>> "Coverity complained; here's a trivial fix" patches in the history is<br>> >>> not very helpful. Ideally, we should have just a handful of such patches<br>> >>> eventually. But it seems we are alread beyond this limit. Please don't<br>> >>> let the pile grow too much.<br>> >>><br>> >>> Thanks,<br>> >>> -- Hannes<br>> >>><br>> >>> _______________________________________________<br>> >>> Cinelerra mailing list<br>> >>> Cinelerra@lists.cinelerra-cv.org<br>> >>> http://lists.cinelerra-cv.org/cgi-bin/mailman/listinfo/cinelerra<br>> >><br>> >> _______________________________________________<br>> >> Cinelerra mailing list<br>> >> Cinelerra@lists.cinelerra-cv.org<br>> >> http://lists.cinelerra-cv.org/cgi-bin/mailman/listinfo/cinelerra<br>> ><br>> > _______________________________________________<br>> > Cinelerra mailing list<br>> > Cinelerra@lists.cinelerra-cv.org<br>> > http://lists.cinelerra-cv.org/cgi-bin/mailman/listinfo/cinelerra<br>> <br>> <br></div>                                    </div></body>
</html>