Re: master 6eeab90632: Don't accept whitespace or hex floats in rgbi: colour specs

[Philip Kaludercic]

Po Lu <luangruo@yahoo.com> writes:

> Philip Kaludercic <philipk@posteo.net> writes:
>
>> It seems the "rgbi" (that stands for RGB Intensity) syntax comes from
>> X11[0], specifically "XCMS" (X Color Management System) Interestingly
>> enough, if you look around you can find that the "rgbi" prefix seems
>> to be case-insensitive[1], while `color-values-from-color-spec' is not:
>>
>>     (color-values-from-color-spec "rgbi:0/0/0") ;; => (0 0 0)
>>     (color-values-from-color-spec "rgbI:0/0/0") ;; => nil
>>     (color-values-from-color-spec "RGBi:0/0/0") ;; => nil
>>     (color-values-from-color-spec "RGBI:0/0/0") ;; => nil
>>
>> The closes thing to a specification I could find was [2], that says:
>>
>>         1.3.2.2.  RGB Intensity      String Specification
>>
>>         An RGB intensity specification in the form of a      string is
>>         identified by the prefix "rgbi:" and whose string conforms
>>         to the following syntax:
>>
>>             rgbi:<red>/<green>/<blue>
>>
>>         Where red, green, and blue are floating      point values between
>>         0.0 and      1.0, inclusive.  The input format for these values
>>         is an optional sign, a string of numbers possibly containing
>>         a decimal point, and an      optional exponent field containing
>>         an E or      e followed by a possibly signed integer string.
>
> See my reply to Mattias, that's not the RGBi specification I'm talking
> about.

The point remains that if rgbi: is supported, it should be
case-insensitive.

-- 
        Philip Kaludercic