Re: patch for bug in colour_fromstring

2015-02-07 Thread Nicholas Marriott
Fixed, thanks On Sat, Feb 07, 2015 at 04:05:56PM -0600, J Raynor wrote: > The colour_fromstring function intends to accept string values ???90" through > ???97", but doesn???t because it checks incorrectly. > > Here???s how it checks ???91???: > > > if (strcasecmp(s, "brightred") == 0

patch for bug in colour_fromstring

2015-02-07 Thread J Raynor
The colour_fromstring function intends to accept string values “90" through “97", but doesn’t because it checks incorrectly.Here’s how it checks “91”: if (strcasecmp(s, "brightred") == 0 ||    (s[0] == '9' && s[1] == '1' && s[1] == '\0')) return (91);Notice it checks s[1] twice, instead of checki