From: Sebastian M. <seb...@un...> - 2008-05-10 21:22:33
|
BTW on this topic: I have started a compatibility matrix at http://vimplugin.org/compatibility Please add your configuration and comments there ... Am Sat, 10 May 2008 10:03:26 +0530 schrieb Nageshwar <nag...@st...>: > I don't know whether the above code works or not. someone, please > check this on linux machine and commit the changes. I tried your code, but it uses again an int instead of long which reintroduces the bug with 64bit systems fixed by "moosydoom" (do we have a policy on realnames !?). Nageshwar, could you try this please? if (Platform.getOS().equals(Platform.OS_LINUX)) { f = c.getField(linuxWID); } else if (Platform.getOS().equals(Platform.OS_WIN32)) { f = c.getField(win32WID); } else { f = c.getField(win32WID); } return f.getLong(parent); ^^^^^^^ This should suffice on 64bit systems. Hmm, the getWid() method doesnt do anything more beside the if statement above. The WidHandler class has only that (static) method. The method is called only at one place. The error handling (return WID_ERROR) is not java-like (thats what exceptions are for..). Would anyone mind if I move the statement to its call at AbstractVimEditor ? Seb. |