Tk Source Code

Attached are the patches to apply to 8.5 HEAD versions
of tkUnixButton.c and tkUnixMenu.c.  These changes
implement TIP 109.

-Brian

Logged In: YES 
user_id=79902

The problem was quite simply a failure to pass the test
suite, and it looked to me like an unexpected case,
somewhere where you thought something would never be NULL
but which could actually happen.  I find the test suite to
be fairly good for finding such stuff.

BTW, it looks good.

Logged In: YES 
user_id=22949

I'm sorry for submitting bad code.  I rushed it trying to
submit the implementation before the conference.  Always a
bad move :-(

I did test the patch across multiple platforms, but in all
cases I did not have a stressed color map.  I fixed this bug
a couple days ago.  There seems to be another bug that is
showing up only on 64-bit AIX5 which is color related, but
is not a crash.  I'm still trying to track this down.  

We will be releasing product by September which will include
this patch, so rest assured that I will wring out any
remaining bugs.

-Brian

Logged In: YES 
user_id=79902

Applied to CVS HEAD.

Brian, I had to alter a few bits to stop the test suite crashing 
with accesses to NULL pointers.  Please could you stress-test 
the version in the core ASAP to check that it is still behaving 
correctly; the problem was that both buttons and menus 
would sometimes try to derive the selectColor argument to 
TkpDrawCheckIndicator() from a NULL pointer, and my fix in 
each case was to force a NULL to get passed through for 
that argument which might not be correct.  If what I have 
done is wrong, please create a new bug and assign it to me.

Logged In: YES 
user_id=79902

I have marginally more free time than Jeff.

File Added - 57200: tip109.tgz

Logged In: YES 
user_id=22949

Fix problems found with previous patch.

File Added - 56672: tip109.patch