Tk Source Code

View Ticket
Login
2009-01-12
06:07 Closed ticket [2496329fff]: Error in bind.test: bad keysym ":" plus 7 other changes artifact: a62c6ec719 user: patthoyts
2009-01-10
05:16 Ticket [2496329fff]: 4 changes artifact: 6ed532d056 user: dgp
04:49 Ticket [2496329fff]: 5 changes artifact: 5dd1bd8b4e user: ferrieux
2009-01-09
19:30 New ticket [2496329fff]. artifact: ea7001f0f8 user: jenglish

Ticket UUID: 2496329
Title: Error in bind.test: bad keysym ":"
Type: Bug Version: None
Submitter: jenglish Created on: 2009-01-09 19:30:35
Subsystem: 01. Bindings Assigned To: patthoyts
Priority: 5 Medium Severity:
Status: Closed Last Modified: 2009-01-12 06:07:03
Resolution: Fixed Closed By: patthoyts
    Closed on: 2009-01-11 23:07:03
Description:
tests/bind.test contains things like:

    event generate .t.f <Key-:>
    event generate .t.f <Key-+>
    event generate .t.f <Key-_>

(found in bind-13.1, a few other places).  This fails with the error 'bad event type or keysym ":"'
X keysyms consist of alphanumeric characters and underscores; see man keysyms(3tk).

Relevant CVS log entries:

| revision 1.24
| date: 2008/11/23 00:47:25;  author: ferrieux;  state: Exp;  lines: +3 -3
| Re-fixed QWERTY/AZERTY sensitivity of key event generation test (thx to Pat Thoyts)
| ----------------------------
| revision 1.23
| date: 2008/11/22 23:45:54;  author: ferrieux;  state: Exp;  lines: +3 -3
| Fixed QWERTY/AZERTY sensitivity of key event generation test

No SF bug # listed, no ChangeLog entry, so no other details available.  I can't imagine that this test ever worked though.
User Comments: patthoyts added on 2009-01-12 06:07:03:
Worked on windows. Now fixed on unix tool. The point of the original patch was that the keysym bindings have a dependency upon the keymap in use and so on non-English versions of Windows (or systems with a non-Querty layout really) many of the bind.test tests were failing. Switching to use symbolic keys works around this issue.

dgp added on 2009-01-10 05:16:31:

see 2336454

ferrieux added on 2009-01-10 04:49:34:
Your CVS history analysis is off by one: those ":" et al. were introduced on rev. 1.25 by Pat Thoyts.
So I'm reassigning to him.

Revision 1.25 - (view) (download) (annotate) - [select for diffs] 
Sun Nov 23 21:40:43 2008 UTC (6 weeks, 5 days ago) by patthoyts 
Branch: MAIN 
Changes since 1.24: +657 -651 lines 
Diff to previous 1.24
Fixed some locale dependencies to reduce the noise on non-English systems