Date: Mon, 09 Nov 2009 21:58:21 +0100 From: =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= <des@des.no> To: Alexander Best <alexbestms@wwu.de> Cc: freebsd-hackers@FreeBSD.org, Giorgos Keramidas <keramida@freebsd.org> Subject: Re: [patch] burncd: honour for envar SPEED Message-ID: <86aayvfmaa.fsf@ds4.des.no> In-Reply-To: <permail-200911091924561e86ffa8000070ba-a_best01@message-id.uni-muenster.de> (Alexander Best's message of "Mon, 09 Nov 2009 20:24:56 %2B0100 (CET)") References: <permail-200911091924561e86ffa8000070ba-a_best01@message-id.uni-muenster.de>
next in thread | previous in thread | raw e-mail | index | archive | help
Alexander Best <alexbestms@wwu.de> writes:
>
> + if ((env_speed = getenv("BURNCD_SPEED")) != NULL) {
> + if (strcasecmp("max", env_speed) == 0)
> + speed = CDR_MAX_SPEED;
> + else
> + speed = atoi(env_speed) * 177;
> + if (speed <= 0)
> + errx(EX_USAGE, "Invalid speed: %s", env_speed);
> + }
> +
> while ((ch = getopt(argc, argv, "def:Flmnpqs:tv")) != -1) {
> switch (ch) {
> case 'd':
You realize you're duplicating 6 lines of non-trivial code for no good
reason?
env_speed = getenv("BURNCD_SPEED");
while ((ch = getopt(...)) != -1) {
switch (ch) {
case 's':
env_speed = optarg;
break;
...
}
}
if (env_speed != NULL) {
if (strcasecmp...) {
...
}
}
DES
--
Dag-Erling Smørgrav - des@des.no
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?86aayvfmaa.fsf>