Issue 56540 - Clipboard code for Windows uses strange/wrong message loop code
Summary: Clipboard code for Windows uses strange/wrong message loop code
Status: ACCEPTED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: OOo 2.0
Hardware: All All
: P3 Trivial with 1 vote (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-10-24 15:52 UTC by carsten.driesner
Modified: 2013-08-07 15:31 UTC (History)
4 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description carsten.driesner 2005-10-24 15:52:30 UTC 
cd->tra: As discussed by phone.

The clipboard code for Windows in the project dtrans uses a strange/wrong
message loop code. Although the comment for the code states that it doesn't
process PostMessage messages it does at the parameter PM_QS_SENDMESSAGE is not
used (only PM_REMOVE). May be this code relates to a crash report, see #126202.
Comment 1 tino.rachui 2005-11-02 14:06:20 UTC 
tra: We probably can get rid of this code completely when the new UNO threading
model is integrated into the Office. So I suggest that we leave things as they
are in this sensitive area until then.
Comment 2 Mathias_Bauer 2006-07-05 11:43:48 UTC 
Carsten, please take over
Comment 3 carsten.driesner 2006-07-10 10:23:15 UTC 
cd: Threading framework will be ready for OOo 3.0, we should take the chance to
fix this issue as we are sure that it's the root of several crashes.
Comment 4 Mathias_Bauer 2007-12-04 16:19:33 UTC 
according to release status meeting -> target 3.x
Comment 5 harveyab 2008-09-08 11:56:00 UTC 
Please see my new comments in Issue 75488

Harvey Block (harveyab)
Comment 6 kpalagin 2008-11-22 19:20:07 UTC 
Carsten,
please consider this issue as it really hurts our users - see all those issues 
listed by Harvey Block in 75488. 
Also the symptom of clipboard being unavailable gets reported on Russian 
community forum quite often.
Please investigate if there is really a problem with our code and if it can be 
fixed for 3.1.

Thanks a lot for your attention!
WBR,
Kirill Palagin.
Comment 7 kpalagin 2008-12-03 14:06:21 UTC 
Carsten, Mathias,
please see if this issue can be fixed for 3.1 (or 3.2) - random delay when 
pasting clipboard really exists. See 75488 for a list of issues which are 
related to this problem. This delay is often reported on Russian community 
forums, which is indication that our uses do suffer from this problem.

Thanks a lot.
WBR,
KP.
Comment 8 Mathias_Bauer 2008-12-03 14:23:06 UTC 
Sorry, no way for 3.1.
Do a query and you will find that we already have more issues with target 3.1
than we can fix.

I also don't see why issuie 75488 and this one should be related. But we lack
the time to find that out.
Comment 9 carsten.driesner 2008-12-03 14:25:02 UTC 
cd: I am not sure if I have enough time to fix this issue. This is a special
code part and making changes without a complete regression test is VERY DANGEROUS!
Comment 10 kpalagin 2008-12-03 14:48:26 UTC 
75488 mught not be related to this one, but the fact that OO is the only app 
experiencing this delay tells me that our code is doing something special with 
the clipboard. And this issue is exactly about strange code.
Any chance to target fot 3.2 (and include new code in first post-3.1 build, so 
that change can be tested well)?

Thanks a lot for your attention.
Comment 11 Mathias_Bauer 2008-12-03 17:16:56 UTC 
Every issue with target 3.x has a chance to get fixed in 3.2. It that will
happen depends on how we prioritize it wrt. all the others we have.