ColourTrans_SetGCOL

[Philip Ludlam]

On 16 Dec, Tony van der Hoff wrote:

>On 16 Dec at 17:16 Philip Ludlam <philip at philipnet.com> wrote in message
><e9d539964e.philip at philipnet.com>
>
>> On 30 Oct, Tony van der Hoff wrote:
>> 
>> > On 28 Oct at 3:22 Christian Ludlam <christian at recoil.org> wrote in
>> > message <5ffcab7c4e.christian at venice.lab>
>> >
>> > > Hi,
>> >> 
>> > > colourtrans_gcol_flags, as used in R3 in colourtrans_set_gcol() should
>> > > have bit 7 meaning "set background colour" as well as the ECF flag in
>> > > bit 8.
>> >
>> > Woo, there's a bug lurking undiscovered since the dawn of time:

[snip]

>> Personally, I think it'll be overkill to append 'Col' to the last two for
>> consistency sake.
>> 
>Well, I wouldn't rename them, I'd just add them as aliases.

Fine then.

>> Does DefMod really treat SetTextCol different to the rest?
>> 
>I thought so, because of the capitalisation. Not so sure now...

Can you find out?


Then fix it (one way or the other) and then issue a .01 release on the
1st/2nd of Jan ;-) .
AFAIK, there's been a few bug fixes this year but no major additions so
I feel a .01 release would flush out the current fixes and give us a new
version for the New Year.


Phil L.
-- 
http://www.philipnet.com/ | http://director.sourceforge.net/
       http://www.windowsadvice.com/blogs/philipnet/

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.