vimplugin-devel

[vimplugin-devel] Problem on windows

[Nageshwar <nag...@st...>]

Hi,

In the file widhandler.java, parent.embeddedHandle was used to get the
window Id. This field is not supported
on eclipse sdk 3.3.2 on windows. I got some errors when I tried to run
the project reporting that the field doesn't exist.

I replaced the line
return parent.embeddedHandle;
with the following.
return c.getField("embeddedHandle").getInt(parent);

I don't know whether the above code works or not. someone, please
check this on linux machine and commit the changes.

-- 
Best Regards,
Nageshwar M.

Re: [vimplugin-devel] Problem on windows

[Sebastian M. <seb...@un...>]

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.