Re: [PATCH weston] Require base-10 for strtol() calls

2016-07-12 Thread Peter Hutterer
On Tue, Jul 12, 2016 at 06:54:49PM -0700, Bryce Harrington wrote: > On Wed, Jul 13, 2016 at 10:27:49AM +1000, Peter Hutterer wrote: > > On Tue, Jul 12, 2016 at 04:51:27PM -0700, Bryce Harrington wrote: > > > The third arg to strtol() specifies the base to assume for the number. > > > When 0 is pass

Re: [PATCH weston] Require base-10 for strtol() calls

2016-07-12 Thread Yong Bakos
On Jul 12, 2016, at 4:51 PM, Bryce Harrington wrote: > > The third arg to strtol() specifies the base to assume for the number. > When 0 is passed, as is currently done in option-parser.c, hexadecimal > and octal numbers are permitted and automatically detected and > converted. > > This change i

Re: [PATCH weston] Require base-10 for strtol() calls

2016-07-12 Thread Bryce Harrington
On Tue, Jul 12, 2016 at 06:49:06PM -0700, Yong Bakos wrote: > On Jul 12, 2016, at 4:51 PM, Bryce Harrington wrote: > > > > The third arg to strtol() specifies the base to assume for the number. > > When 0 is passed, as is currently done in option-parser.c, hexadecimal > > and octal numbers are pe

Re: [PATCH weston] Require base-10 for strtol() calls

2016-07-12 Thread Bryce Harrington
On Wed, Jul 13, 2016 at 10:27:49AM +1000, Peter Hutterer wrote: > On Tue, Jul 12, 2016 at 04:51:27PM -0700, Bryce Harrington wrote: > > The third arg to strtol() specifies the base to assume for the number. > > When 0 is passed, as is currently done in option-parser.c, hexadecimal > > and octal num

Re: [PATCH weston] Require base-10 for strtol() calls

2016-07-12 Thread Peter Hutterer
On Tue, Jul 12, 2016 at 04:51:27PM -0700, Bryce Harrington wrote: > The third arg to strtol() specifies the base to assume for the number. > When 0 is passed, as is currently done in option-parser.c, hexadecimal > and octal numbers are permitted and automatically detected and > converted. > > This

[PATCH weston] Require base-10 for strtol() calls

2016-07-12 Thread Bryce Harrington
The third arg to strtol() specifies the base to assume for the number. When 0 is passed, as is currently done in option-parser.c, hexadecimal and octal numbers are permitted and automatically detected and converted. This change is an expansion of f6051cbab84c0e577473b67f0585c0f329eb80fe to cover t