From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Date: Mon, 06 Apr 2009 09:36:31 +0000 Subject: Re: [lm-sensors] [PATCH 0/8] sensord: Some cleanups, Message-Id: <20090406113631.02483a62@hyperion.delvare> List-Id: References: <20090406075553.GA4734@ubuntu> In-Reply-To: <20090406075553.GA4734@ubuntu> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org Hi Andre, On Mon, 6 Apr 2009 09:55:53 +0200, Andre Prendel wrote: > Hello Jean, > > this patch series fixes some issues caused by the coding style > conversion a while ago. Most of them are deep indentation levels and > the related long lines. > > Another patch removes the hidden commandline interface. You can call > sensord as an console application if you have a link (sensors) > pointing to the sensors deamon (sensord). IMO that's really > intransparent and I don't know any reason using this. > > Further there is a patch introducing a new structure > (sensord_arguments). The extern struct sensord_args in args.c now is > the place for all the commandline arguments. > > A last one replaces the deprecated signal() function by sigaction. Thanks for doing all this. Please note that I am pretty busy with kernel work these days so it will take some time before I can find the time to review your changes. Thanks for your patience :) -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors