From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Date: Tue, 16 Jun 2009 11:54:08 +0000 Subject: Re: [lm-sensors] [PATCH 0/5] sensord: Cleanup of the sensors deamon Message-Id: <20090616135408.581b1ddf@hyperion.delvare> List-Id: References: <20090615074546.GD4667@ubuntu> In-Reply-To: <20090615074546.GD4667@ubuntu> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Mon, 15 Jun 2009 09:45:46 +0200, Andre Prendel wrote: > Hi Jean, > > here is another patch series cleaning up the sensors deamon. > > Some patches fixing some coding style issues, breaking long lines and > making code more readable by the refactoring of big functions. > > Another one removes some needless casts and there is a patch > converting the HTML code (CGI script) generated by sensord to lower > case. So we're closer to XHTML. Unfortunately the output of rrdcgi > isn't very strict. > > With this series I've finished the cleanup caused by the coding style > conversion. During the work I saw some things to fix, so I will go on > hacking the deamon. Now I hope the code is more readable and the > maintenance and development should be easier. BTW I don't like some > details of the implementation (see notes in patch 4/5). Some stuff > looks needless. Further I would like it if the deamon would be closer > to the sensors console tool. This would ease maintenance too. So you > see what I'm going to do in the future. Very nice. Thanks a lot for working on this, this was very needed. > This here is post 3.1.1 material. Maybe we could make a stable branch > 3.1.1? Otherwise we wait with the commits until 3.1.1 is released. Let's just wait until 3.1.1 is released, it is less than 1 week away now and nobody reviewed your patches yet anyway. I don't think it makes much sense to create branches for 1-week SVN freeze periods. And anyway, during these periods, we should all be testing the code rather that developing new stuff ;) -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors