svn commit: r199079 - head/sbin/camcontrol

[Nathan Whitehorn]

Alexander Motin wrote:
> Author: mav
> Date: Mon Nov  9 11:39:51 2009
> New Revision: 199079
> URL: http://svn.freebsd.org/changeset/base/199079
>
> Log:
>   Add support for ATA Power Management.
>
> Modified:
>   head/sbin/camcontrol/camcontrol.8
>   head/sbin/camcontrol/camcontrol.c
>   
[...]
> +static int
> +atapm(struct cam_device *device, int argc, char **argv,
> +		 char *combinedopt, int retry_count, int timeout)
> +{
> +	union ccb *ccb;
> +	int retval = 0;
> +	int t = -1;
> +	char c;
>
>   
[...]
> +
> +	while ((c = getopt(argc, argv, combinedopt)) != -1) {
> +		switch (c) {
> +		case 't':
> +			t = atoi(optarg);
> +			break;
> +		default:
> +			break;
> +		}
> +	}
This broke world on PowerPC, where char is unsigned. Could you change 
the type of c to int, or explicitly mark it signed?
-Nathan