From: Sami Kerola <kerolasa@iki.fi>
To: util-linux@vger.kernel.org
Cc: kerolasa@iki.fi
Subject: [PATCH 01/15] cytune: rename threshold and timeout variables
Date: Sun, 4 May 2014 16:49:42 +0100 [thread overview]
Message-ID: <1399218596-21321-2-git-send-email-kerolasa@iki.fi> (raw)
In-Reply-To: <1399218596-21321-1-git-send-email-kerolasa@iki.fi>
Especially the treshold was difficult to read, as the variable name had
no reference what it was setting. Effect of the variables
s/get/get_current/ and s/get_def/get_defaults/ seem to documented wrong
way, that became apparent after looking into this name space.
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
---
sys-utils/cytune.c | 101 +++++++++++++++++++++++++++--------------------------
1 file changed, 52 insertions(+), 49 deletions(-)
diff --git a/sys-utils/cytune.c b/sys-utils/cytune.c
index 8a42f96..544865f 100644
--- a/sys-utils/cytune.c
+++ b/sys-utils/cytune.c
@@ -274,23 +274,26 @@ int main(int argc, char **argv)
{
int query = 0;
int interval = 1;
- int set = 0;
- int set_val = -1;
- int get = 0;
- int set_def = 0;
- int set_def_val = -1;
- int get_def = 0;
- int set_time = 0;
- int set_time_val = -1;
- int set_def_time = 0;
- int set_def_time_val = -1;
int errflg = 0;
int file;
int numfiles;
int i;
+
+ int get_current = 0;
+ int get_defaults = 0;
unsigned long threshold_value;
unsigned long timeout_value;
+ int set_threshold = 0;
+ int threshold_val = -1;
+ int set_threshold_def = 0;
+ int threshold_def_val = -1;
+
+ int set_timeout = 0;
+ int timeout_val = -1;
+ int set_timeout_def = 0;
+ int timeout_def_val = -1;
+
static const struct option longopts[] = {
{"set-threshold", required_argument, NULL, 's'},
{"get-threshold", no_argument, NULL, 'g'},
@@ -331,48 +334,48 @@ int main(int argc, char **argv)
}
break;
case 's':
- ++set;
- set_val = strtou32_or_err(optarg, _("Invalid set value"));
- if (set_val < 1 || 12 < set_val) {
- warnx(_("Invalid set value: %d"), set_val);
+ ++set_threshold;
+ threshold_val = strtou32_or_err(optarg, _("Invalid threshold value"));
+ if (threshold_val < 1 || 12 < threshold_val) {
+ warnx(_("Invalid threshold value: %d"), threshold_val);
errflg++;
}
break;
case 'S':
- ++set_def;
- set_def_val = strtou32_or_err(optarg,
- _("Invalid default value"));
- if (set_def_val < 0 || 12 < set_def_val) {
- warnx(_("Invalid default value: %d"),
- set_def_val);
+ ++set_threshold_def;
+ threshold_def_val = strtou32_or_err(optarg,
+ _("Invalid threshold default value"));
+ if (threshold_def_val < 0 || 12 < threshold_def_val) {
+ warnx(_("Invalid threshold default value: %d"),
+ threshold_def_val);
errflg++;
}
break;
case 't':
- ++set_time;
- set_time_val = strtou32_or_err(optarg,
- _("Invalid set time value"));
- if (set_time_val < 1 || 255 < set_time_val) {
- warnx(_("Invalid set time value: %d"),
- set_time_val);
+ ++set_timeout;
+ timeout_val = strtou32_or_err(optarg,
+ _("Invalid set timeout value"));
+ if (timeout_val < 1 || 255 < timeout_val) {
+ warnx(_("Invalid set timeout value: %d"),
+ timeout_val);
errflg++;
}
break;
case 'T':
- ++set_def_time;
- set_def_time_val = strtou32_or_err(optarg,
- _("Invalid default time value"));
- if (set_def_time_val < 0 || 255 < set_def_time_val) {
+ ++set_threshold_def;
+ timeout_def_val = strtou32_or_err(optarg,
+ _("Invalid default timeout value"));
+ if (timeout_def_val < 0 || 255 < timeout_def_val) {
warnx(_("Invalid default time value: %d"),
- set_def_time_val);
+ timeout_def_val);
errflg++;
}
break;
case 'g':
- ++get;
+ ++get_current;
break;
case 'G':
- ++get_def;
+ ++get_defaults;
break;
case 'V':
printf(_("%s from %s\n"), program_invocation_short_name,
@@ -388,62 +391,62 @@ int main(int argc, char **argv)
if (errflg
|| (numfiles == 0)
- || (!query && !set && !set_def && !get && !get_def && !set_time && !set_def_time)
- || (set && set_def)
- || (set_time && set_def_time)
- || (get && get_def))
+ || (!query && !set_threshold && !set_threshold_def && !get_defaults && !get_current && !set_timeout && !set_timeout_def)
+ || (set_threshold && set_threshold_def)
+ || (set_timeout && set_timeout_def)
+ || (get_defaults && get_current))
usage(stderr);
/* For signal routine. */
global_optind = optind;
- if (set || set_def) {
+ if (set_threshold || set_threshold_def) {
for (i = optind; i < argc; i++) {
file = open(argv[i], O_RDONLY);
if (file == -1)
err(EXIT_FAILURE, _("cannot open %s"), argv[i]);
if (ioctl(file,
- set ? CYSETTHRESH : CYSETDEFTHRESH,
- set ? set_val : set_def_val))
+ set_threshold ? CYSETTHRESH : CYSETDEFTHRESH,
+ set_threshold ? threshold_val : threshold_def_val))
err(EXIT_FAILURE,
_("cannot set %s to threshold %d"), argv[i],
- set ? set_val : set_def_val);
+ set_threshold ? threshold_val : threshold_def_val);
close(file);
}
}
- if (set_time || set_def_time) {
+ if (set_timeout || set_timeout_def) {
for (i = optind; i < argc; i++) {
file = open(argv[i], O_RDONLY);
if (file == -1)
err(EXIT_FAILURE, _("cannot open %s"), argv[i]);
if (ioctl(file,
- set_time ? CYSETTIMEOUT : CYSETDEFTIMEOUT,
- set_time ? set_time_val : set_def_time_val))
+ set_timeout ? CYSETTIMEOUT : CYSETDEFTIMEOUT,
+ set_timeout ? timeout_val : timeout_def_val))
err(EXIT_FAILURE,
_("cannot set %s to time threshold %d"),
argv[i],
- set_time ? set_time_val : set_def_time_val);
+ set_timeout ? timeout_val : timeout_def_val);
close(file);
}
}
- if (get || get_def) {
+ if (get_defaults || get_current) {
for (i = optind; i < argc; i++) {
file = open(argv[i], O_RDONLY);
if (file == -1)
err(EXIT_FAILURE, _("cannot open %s"), argv[i]);
if (ioctl
- (file, get ? CYGETTHRESH : CYGETDEFTHRESH,
+ (file, get_defaults ? CYGETTHRESH : CYGETDEFTHRESH,
&threshold_value))
err(EXIT_FAILURE,
_("cannot get threshold for %s"), argv[i]);
if (ioctl
- (file, get ? CYGETTIMEOUT : CYGETDEFTIMEOUT,
+ (file, get_current ? CYGETTIMEOUT : CYGETDEFTIMEOUT,
&timeout_value))
err(EXIT_FAILURE,
_("cannot get timeout for %s"), argv[i]);
close(file);
- if (get)
+ if (get_defaults)
printf(_("%s: %ld current threshold and %ld current timeout\n"),
argv[i], threshold_value, timeout_value);
else
--
1.9.2
next prev parent reply other threads:[~2014-05-04 15:50 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-04 15:49 [PATCH 00/15] cytune modernization Sami Kerola
2014-05-04 15:49 ` Sami Kerola [this message]
2014-05-05 8:34 ` [PATCH 01/15] cytune: rename threshold and timeout variables Benno Schulenberg
2014-05-04 15:49 ` [PATCH 02/15] cytune: remove unnecessary variables Sami Kerola
2014-05-05 8:29 ` Benno Schulenberg
2014-05-04 15:49 ` [PATCH 03/15] cytune: be consistent with interval data type Sami Kerola
2014-05-04 15:49 ` [PATCH 04/15] cytune: use single loop for setting and getting ioctl() calls Sami Kerola
2014-05-05 8:24 ` Benno Schulenberg
2014-05-04 15:49 ` [PATCH 05/15] cytune: kernel still does not have send_count in cyclades_monitor structure Sami Kerola
2014-05-04 15:49 ` [PATCH 06/15] cytune: prefer sigaction(), and remove unnecessary abstractions Sami Kerola
2014-05-04 15:49 ` [PATCH 07/15] cytune: add cyg_get_mon() to avoid duplicate code Sami Kerola
2014-05-04 15:49 ` [PATCH 08/15] cytune: add structure to hold run time configuration Sami Kerola
2014-05-04 15:49 ` [PATCH 09/15] cytune: pull signal handling and statistic priting apart Sami Kerola
2014-05-04 15:49 ` [PATCH 10/15] cytune: remove unnecessary type casts Sami Kerola
2014-05-04 15:49 ` [PATCH 11/15] cytune: deprecate undescriptive options Sami Kerola
2014-05-04 15:49 ` [PATCH 12/15] cytune: add filename to struct cyclades_control Sami Kerola
2014-05-04 15:49 ` [PATCH 13/15] cytune: add noreturn function attributes Sami Kerola
2014-05-04 15:49 ` [PATCH 14/15] cytune: use matching type in struct cyclades_control with kernel Sami Kerola
2014-05-04 15:49 ` [PATCH 15/15] cytune: update copyright Sami Kerola
2014-05-07 7:10 ` [PATCH 00/15] cytune modernization Karel Zak
2014-05-07 8:12 ` Sami Kerola
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=1399218596-21321-2-git-send-email-kerolasa@iki.fi \
--to=kerolasa@iki.fi \
--cc=util-linux@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox