At 05:02 PM 12/22/00 +0800, you wrote:
>X11 clipboard driver should call only exported win32 functions IMO.
why only the clipboard driver ? To which use ? Do you want to use native
user ? There are other files to clean in this case. Lot of work.
>Driver doesn't need to know about internal clipboard cache at all.
>All it should do is to provide a way of setting clipboard data
>to the X buffer and a way of getting data from it. Not more not less.
That's nice but it means a total rewrite of the file. Internals are used
everywhere. Btw CLIPBOARD_IsPresent is called at two other places
in the file - I checked this before sending the patch. This file will be
completely rewritten some day when some communication between
processes is working again and a true clipboard can exchange objects
between Wine apps. Something for which there is no roadmap, no delay,
nothing - 3 months ? 6 months ? a year ? I have not the slightest idea.
I don't see a need to do it now, when it will buy nothing significant. I am
not even qualified to do it anyway :-). I just want it to work in the limited
way it can work now.
>Anyway when driver is called, CLIPBOARD_IsPresent should be already
>tested.
As the driver is now, this routine is responsible for *filling* the cache. I don't
see how to test if there is data in the cache between filling it. If I try, I expect
that the clipboard will always be found to be empty - just what is happening now.
Gerard
