All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andre Prendel <andre.prendel@gmx.de>
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] [PATCH 0/5] sensord: Cleanup of the sensors deamon
Date: Mon, 15 Jun 2009 07:45:46 +0000	[thread overview]
Message-ID: <20090615074546.GD4667@ubuntu> (raw)

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.

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.

Thanks,
Andre
---

 rrd.c   |  140 +++++++++++++++++++++++++++----------------
 sense.c |  204
 ++++++++++++++++++++++++++++++++++++++--------------------------
 2 files changed, 210 insertions(+), 134 deletions(-)

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

             reply	other threads:[~2009-06-15  7:45 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-15  7:45 Andre Prendel [this message]
2009-06-16 11:54 ` [lm-sensors] [PATCH 0/5] sensord: Cleanup of the sensors deamon Jean Delvare
2009-10-03  8:35 ` Jean Delvare
2009-10-04 19:07 ` Andre Prendel
2009-10-05  9:25 ` Jean Delvare

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090615074546.GD4667@ubuntu \
    --to=andre.prendel@gmx.de \
    --cc=lm-sensors@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.