On 7/25/12 7:44 PM, Konke Radlow wrote:
> --- /dev/null
> +++ b/utils/rds-ctl/rds-ctl.cpp
> @@ -0,0 +1,978 @@
> +/*
> + * rds-ctl.cpp is based on v4l2-ctl.cpp
> + *
> + * the following applies for all RDS related parts:
> + * Copyright 2012 Cisco Systems, Inc. and/or its affiliates. All rights reserved.
> + * Author: Konke Radlow <koradlow@xxxxxxxxx>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU Lesser General Public License as published by
> + * the Free Software Foundation; either version 2.1 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Suite 500, Boston, MA 02110-1335 USA
> + */
> +
> +#include <unistd.h>
> +#include <stdlib.h>
> +#include <stdio.h>
> +#include <string.h>
> +#include <wchar.h>
> +#include <locale.h>
> +#include <inttypes.h>
> +#include <getopt.h>
> +#include <sys/types.h>
> +#include <fcntl.h>
> +#include <errno.h>
> +#include <sys/ioctl.h>
> +#include <sys/time.h>
> +#include <dirent.h>
> +#include <config.h>
> +#include <signal.h>
> +
> +#ifdef HAVE_SYS_KLOG_H
> +#include <sys/klog.h>
> +#endif
You don't call klog, so you can drop these three lines
> +static int parse_cl(int argc, char **argv)
> +{
> + int i = 0;
> + int idx = 0;
> + int opt = 0;
> + char short_options[26 * 2 * 2 + 1];
> +
> + if (argc == 1) {
> + usage_hint();
> + exit(1);
> + }
> + for (i = 0; long_options[i].name; i++) {
> + if (!isalpha(long_options[i].val))
> + continue;
> + short_options[idx++] = long_options[i].val;
> + if (long_options[i].has_arg == required_argument)
> + short_options[idx++] = ':';
> + }
> + while (1) {
> + // TODO: remove option_index ?
> + int option_index = 0;
> +
> + short_options[idx] = 0;
> + opt = getopt_long(argc, argv, short_options,
> + long_options, &option_index);
> + if (opt == -1)
> + break;
> +
> + params.options[(int)opt] = 1;
> + switch (opt) {
> + case OptSetDevice:
> + strncpy(params.fd_name, optarg, 80);
> + if (optarg[0] >= '0' && optarg[0] <= '9' && optarg[1] == 0) {
see isdigit from <types.h> (or std::isdigit from <ctypes>)
Thanks,
Gregor
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
