Hi Jean-Pierre, On 09 Dec 2018 18:04, jp charras wrote:> Le 08/12/2018 à 22:38, Robert Abel a écrit : >> Wire color was parsed using parseInt alone. >> However, input contains a comma after every color component: >> rgb(red, green, blue) >> > > However I committed a more basic patch. I thought about implementing it like in your patch as well. However, now you allow very ill-formatted inputs as well and still won't catch if parseInt doesn't work.
Consider rgb(5,,, ,,,6,,,, ;7) No error message will be shown and blue will still be dropped. Regards, Robert _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : kicad-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
