summaryrefslogtreecommitdiff
path: root/miscutils/hdparm.c
diff options
context:
space:
mode:
authorRob Landley2006-05-08 15:35:46 +0000
committerRob Landley2006-05-08 15:35:46 +0000
commit19802563ed14d75674ba863ee2b5ec29df539abb (patch)
tree34e9dadec958a0e853d10f1a38593285f0228de8 /miscutils/hdparm.c
parent0ea0aefcb5cf4e0a589bda0b0b78a1c39f5f9c10 (diff)
downloadbusybox-19802563ed14d75674ba863ee2b5ec29df539abb.zip
busybox-19802563ed14d75674ba863ee2b5ec29df539abb.tar.gz
The if() is cheaper than the extra function parameter. According to make
bloatcheck, this saves 112 bytes on x86...
Diffstat (limited to 'miscutils/hdparm.c')
-rw-r--r--miscutils/hdparm.c52
1 files changed, 26 insertions, 26 deletions
diff --git a/miscutils/hdparm.c b/miscutils/hdparm.c
index e099c2b..3c0bcc9 100644
--- a/miscutils/hdparm.c
+++ b/miscutils/hdparm.c
@@ -2298,9 +2298,8 @@ static void identify_from_stdin(void)
#endif
/* busybox specific stuff */
-static void parse_opts(int flag, unsigned long *get, unsigned long *set, unsigned long *value, int min, int max)
+static void parse_opts(unsigned long *get, unsigned long *set, unsigned long *value, int min, int max)
{
- if (flag) {
/* noisy is a global var */
if (get) { /* *get is initialized to 0 */
*get = noisy;
@@ -2310,7 +2309,6 @@ static void parse_opts(int flag, unsigned long *get, unsigned long *set, unsigne
*set = 1;
*value = bb_xgetlarg(optarg, 10, min, max);
}
- }
}
#ifdef HDIO_DRIVE_CMD
@@ -2397,47 +2395,49 @@ int hdparm_main(int argc, char **argv)
quiet = 1;
noisy = 0;
}
- parse_opts((c == 'u'), &get_unmask, &set_unmask, &unmask, 0, 1);
- USE_FEATURE_HDPARM_HDIO_GETSET_DMA(parse_opts((c == 'd'), &get_dma, &set_dma, &dma, 0, 9));
- parse_opts((c == 'n'), &get_nowerr, &set_nowerr, &nowerr, 0, 1);
- parse_opts_v3((c == 'p'), &noisy_piomode, &set_piomode, &piomode);
- parse_opts((c == 'r'), &get_readonly, &set_readonly, &readonly, 0, 1);
- parse_opts((c == 'm'), &get_mult, &set_mult, &mult, 0, INT_MAX /*32*/);
- parse_opts((c == 'c'), &get_io32bit, &set_io32bit, &io32bit, 0, INT_MAX /*8*/);
- parse_opts((c == 'k'), &get_keep, &set_keep, &keep, 0, 1);
- parse_opts((c == 'a'), &get_readahead, &set_readahead, &Xreadahead, 0, INT_MAX);
- parse_opts((c == 'B'), &get_apmmode, &set_apmmode, &apmmode, 1, 255);
+ if (c == 'u') parse_opts(&get_unmask, &set_unmask, &unmask, 0, 1);
+ USE_FEATURE_HDPARM_HDIO_GETSET_DMA(if (c == 'd') parse_opts(&get_dma, &set_dma, &dma, 0, 9));
+ if (c == 'n') parse_opts(&get_nowerr, &set_nowerr, &nowerr, 0, 1);
+ parse_opts_v3((c == 'p'),&noisy_piomode, &set_piomode, &piomode);
+ if (c == 'r') parse_opts(&get_readonly, &set_readonly, &readonly, 0, 1);
+ if (c == 'm') parse_opts(&get_mult, &set_mult, &mult, 0, INT_MAX /*32*/);
+ if (c == 'c') parse_opts(&get_io32bit, &set_io32bit, &io32bit, 0, INT_MAX /*8*/);
+ if (c == 'k') parse_opts(&get_keep, &set_keep, &keep, 0, 1);
+ if (c == 'a') parse_opts(&get_readahead, &set_readahead, &Xreadahead, 0, INT_MAX);
+ if (c == 'B') parse_opts(&get_apmmode, &set_apmmode, &apmmode, 1, 255);
do_flush |= do_timings = (c == 't');
do_flush |= do_ctimings = (c == 'T');
#ifdef HDIO_DRIVE_CMD
- parse_opts((c == 'S'), &get_standby, &set_standby, &standby_requested, 0, INT_MAX);
- parse_opts((c == 'D'), &get_defects, &set_defects, &defects, 0, INT_MAX);
- parse_opts((c == 'P'), &get_prefetch, &set_prefetch, &prefetch, 0, INT_MAX);
+ if (c == 'S') parse_opts(&get_standby, &set_standby, &standby_requested, 0, INT_MAX);
+ if (c == 'D') parse_opts(&get_defects, &set_defects, &defects, 0, INT_MAX);
+ if (c == 'P') parse_opts(&get_prefetch, &set_prefetch, &prefetch, 0, INT_MAX);
parse_opts_v3((c == 'X'), &get_xfermode, &set_xfermode, &xfermode_requested);
- parse_opts((c == 'K'), &get_dkeep, &set_dkeep, &prefetch, 0, 1);
- parse_opts((c == 'A'), &get_lookahead, &set_lookahead, &lookahead, 0, 1);
- parse_opts((c == 'L'), &get_doorlock, &set_doorlock, &doorlock, 0, 1);
- parse_opts((c == 'W'), &get_wcache, &set_wcache, &wcache, 0, 1);
+ if (c == 'K') parse_opts(&get_dkeep, &set_dkeep, &prefetch, 0, 1);
+ if (c == 'A') parse_opts(&get_lookahead, &set_lookahead, &lookahead, 0, 1);
+ if (c == 'L') parse_opts(&get_doorlock, &set_doorlock, &doorlock, 0, 1);
+ if (c == 'W') parse_opts(&get_wcache, &set_wcache, &wcache, 0, 1);
parse_opts_v3((c == 'C'), &get_powermode, NULL, NULL);
parse_opts_v2((c == 'y'), &get_standbynow, &set_standbynow);
parse_opts_v2((c == 'Y'), &get_sleepnow, &set_sleepnow);
reread_partn = (c == 'z');
parse_opts_v2((c == 'Z'), &get_seagate, &set_seagate);
#endif
- USE_FEATURE_HDPARM_HDIO_UNREGISTER_HWIF(parse_opts((c == 'U'), NULL, &unregister_hwif, &hwif, 0, INT_MAX));
+ USE_FEATURE_HDPARM_HDIO_UNREGISTER_HWIF(if (c == 'U') parse_opts(NULL, &unregister_hwif, &hwif, 0, INT_MAX));
#ifdef HDIO_GET_QDMA
+ if (c == 'Q') {
#ifdef HDIO_SET_QDMA
- parse_opts((c == 'Q'), &get_dma_q, &set_dma_q, &dma_q, 0, INT_MAX);
+ parse_opts(&get_dma_q, &set_dma_q, &dma_q, 0, INT_MAX);
#else
- parse_opts((c == 'Q'), &get_dma_q, NULL, NULL, 0, 0);
+ parse_opts(&get_dma_q, NULL, NULL, 0, 0);
#endif
+ }
#endif
USE_FEATURE_HDPARM_HDIO_DRIVE_RESET( perform_reset = (c == 'r'));
- USE_FEATURE_HDPARM_HDIO_TRISTATE_HWIF(parse_opts((c == 'x'), NULL, &perform_tristate, &tristate, 0, 1));
- USE_FEATURE_HDPARM_HDIO_TRISTATE_HWIF(parse_opts((c == 'b'), &get_busstate, &set_busstate, &busstate, 0, 2));
+ USE_FEATURE_HDPARM_HDIO_TRISTATE_HWIF(if (c == 'x') parse_opts(NULL, &perform_tristate, &tristate, 0, 1));
+ USE_FEATURE_HDPARM_HDIO_TRISTATE_HWIF(if (c == 'b') parse_opts(&get_busstate, &set_busstate, &busstate, 0, 2));
#if ENABLE_FEATURE_HDPARM_HDIO_SCAN_HWIF
if (c == 'R') {
- parse_opts((c == 'R'), NULL, &scan_hwif, &hwif_data, 0, INT_MAX);
+ parse_opts(NULL, &scan_hwif, &hwif_data, 0, INT_MAX);
hwif_ctrl = bb_xgetlarg((argv[optind]) ? argv[optind] : "", 10, 0, INT_MAX);
hwif_irq = bb_xgetlarg((argv[optind+1]) ? argv[optind+1] : "", 10, 0, INT_MAX);
/* Move past the 2 additional arguments */