[Xquartz-dev] 2.3.2_beta3

[Peter Dyballa]

Am 16.11.2008 um 16:00 schrieb Jeremy Huddleston:

> Whoops... I just noticed this in the patch... you should undo that
> change.
> -                    last_key_layout = key_layout;
>                       CFRelease(clear);
> +                    last_key_layout = key_layout;


Right now I have in X11Application.m:

  1030	            if(darwinSyncKeymap) {
  1031	#if MAC_OS_X_VERSION_MIN_REQUIRED >= 1050
  1032	                TISInputSourceRef key_layout =  
TISCopyCurrentKeyboardLayoutInputSource();
  1033	                TISInputSourceRef clear;
  1034	                if (CFEqual(key_layout, last_key_layout)) {
  1035	                    CFRelease(key_layout);
  1036	                } else {
  1037	                    /* Swap/free thread-safely */
  1038	                    clear = last_key_layout;
  1039	                    CFRelease(clear);
  1040	                    last_key_layout = key_layout;
  1041	#else
  1042	                KeyboardLayoutRef key_layout;
  1043	                KLGetCurrentKeyboardLayout(&key_layout);
  1044	                if(key_layout != last_key_layout) {
  1045	                    last_key_layout = key_layout;
  1046	#endif

and I should move line #1040 before line #1039, as it was before?  
(Anyway, since MAC_OS_X_VERSION_MIN_REQUIRED is for me probably 1040  
it would not make a difference for me right now, and Leopard might  
arrive at Christmas.)

--
Greetings

   Pete

"Evolution"            o           __o                     _o _
           °\___o      /0~         -\<,              ^\___ /=\\_/-%
oo~_______ /\ /\______/ \_________O/ O_______________o===>-->O--o____