On Mon, May 20, 2013 at 09:04:25PM -0300, Rafael Aquini wrote:
> - while ((c = getopt_long(argc, argv, "ahdefp:svVL:U:",
> + while ((c = getopt_long(argc, argv, "ahcdefp:svVL:U:",
> long_opts, NULL)) != -1) {
> switch (c) {
> case 'a': /* all */
> @@ -738,8 +753,11 @@ int main(int argc, char *argv[])
> case 'U':
> add_uuid(optarg);
> break;
> + case 'c':
> + discard += 2;
> + break;
> case 'd':
> - discard = 1;
> + discard += 1;
this is fragile, it would be better to use
case 'c':
discard |= SWAP_FLAG_DISCARD_CLUSTER;
break;
case 'd':
discard |= SWAP_FLAG_DISCARD;
break;
and use directly the flags everywhere in the code than use magical
numbers '1' and '2' etc.
Karel
--
Karel Zak <k...@redhat.com>
http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
