[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User pl changed the following: What|Old value |New value Status|NEW |STARTED Target milestone|OOo 3.0 |OOo 2.0.4 --- Additional comments from [EMAIL PROTECTED] Tue Apr 4 04:31:46 -0700 2006 --- we should consider this for 2.0.4 - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User pl changed the following: What|Old value |New value CC|'hdu,mhu,mt' |'' Issue type|PATCH |ENHANCEMENT --- Additional comments from [EMAIL PROTECTED] Tue Apr 4 08:11:19 -0700 2006 --- change type to enhancement - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User pl changed the following: What|Old value |New value Target milestone|OOo Later |OOo 3.0 --- Additional comments from [EMAIL PROTECTED] Fri Dec 9 04:59:57 -0800 2005 --- set target - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User ssa changed the following: What|Old value |New value CC|'mhu,mt' |'hdu,mhu,mt' Assigned to|ssa |pl --- Additional comments from [EMAIL PROTECTED] Mon Dec 5 01:56:43 -0800 2005 --- ssa-pl: please take over, you were already involved... - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 --- Additional comments from [EMAIL PROTECTED] Fri Jul 29 02:28:04 -0700 2005 --- I'd like to accept the patch regarding the event processing but: - this change schould be made on all platforms, not gtk only - we should clean up the code where reschedule is already called in a loop - I have a bad feeling without some explicit exit condition for the while loop (may be we should stop after 1000 calls to not block this thread) - unless I see a drastical improvement in rendering speed or responsiveness I would retarget the patch to the next major release, this gives us enough time to check it on all platforms, including multi threading scenarios like API test, Java wizards etc. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User mmeeks changed the following: What|Old value |New value Target milestone|OOo 2.0.1 |OOo Later --- Additional comments from [EMAIL PROTECTED] Fri Jul 29 05:16:06 -0700 2005 --- sounds reasonable - really just wanted to push it onto someone else's queue ;-) set to OOoLater. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User mmeeks changed the following: What|Old value |New value Assigned to|mmeeks|ssa Status|REOPENED |NEW Target milestone|OOo 2.0 |OOo 2.0.1 --- Additional comments from [EMAIL PROTECTED] Thu Jul 28 03:10:50 -0700 2005 --- So - we've been shipping with this patch for a while - and - see no related breakage, AFAIR it should improve re-rendering performance, but ... ;-) you really don't want to accept it ? - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 --- Additional comments from [EMAIL PROTECTED] Thu May 19 04:21:53 -0700 2005 --- AS-mmeeks: As I already said ... your patch isnt the right one. It can produce other issues. But PL point me to another task ... #48550# .. wich may be can help to fix your problem. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 --- Additional comments from [EMAIL PROTECTED] Mon Mar 21 07:45:15 -0800 2005 --- do we come to an agreement here ? - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User mmeeks changed the following: What|Old value |New value Status|RESOLVED |REOPENED Resolution|INVALID | --- Additional comments from [EMAIL PROTECTED] Wed Feb 9 02:05:03 -0800 2005 --- as: ah - since I'm using m76 I didn't see your nice sleeper thread thing to fix this :-) that looks substantially better. So the speed-status-bar.diff patch is now obsolete for m76+. However I'm less convinced the vcl patch is. I'm still not convinced that 40 times/second is going to give a responsive GUI behavior, of course, you've tried it - but in my recent tests, after the 1st second, without the above patch responsiveness in other windows was dire - that was processing one event every 100ms. To give you some idea; when eg. an expose event comes in, gtk+ handles it, tries to collate it with other events (for efficiency) / duplicate elimination, so - there is a priority scheme. What this means is that we will process one incoming X event per 40ms, but - we will never hit 'idle' - ie. we will never process the much lower priority 'idle' handler that does the work of aggregating exposes, and actually dispatching the redraw events to VCL [ which then of course, delays them via another idle handler AFAIR ;-]. ie. if the X server is providing more than 25 X events per second - without the above patch we'll never actually re-render :-) ie. I think this is a real problem for interactivity, particularly with the gtk+ backend (which it affects). - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 --- Additional comments from [EMAIL PROTECTED] Tue Feb 8 00:44:21 -0800 2005 --- adding mt for CC; performance improvement - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User as changed the following: What|Old value |New value Assigned to|as|mmeeks --- Additional comments from [EMAIL PROTECTED] Tue Feb 8 22:24:13 -0800 2005 --- AS-PL: My fix was not related to speed up calc loading only. We (MHU and I) adressed the following problems: a) decreasing the amount of system calls. The old implementation used clock() provided by the system header time.h to check weather a reschedule call should be done or not. And this check (which can be very expensive on unix) was done for every of the more then 65.000 setValue() calls from the calc filter. = Now we use a timed-wait-condition to prevent such calls. b) The old implementation supressed any reschedule call for the first second of the live time of a progress. After that 1 second EVERY setvalue() call (... 65.000 for calc ...) forces 1 Application::Reschedule(). So there was no relation between the count of Reschedule() and a special time frame. = Now Reschedule() is bound to a special time frame. It's called every 25 ms ... ~40 times per second and does not depend from count of setValue() calls anymore. From my point of view that should be enough. Because on the other side we try to speed up loading of documents ... not to increase the count of executed events. AS-mmeeks: Sorry - but your patch isn't the right one. Pleae have a look on the actual master version of the statusindicatorfactory.cxx file [rev. 1.13]. As you can see ... the methods clock(), impl_get10ThSec() are not used any longer. From now the count of Reschedule() within the progress should be fix and not depending from the using of such progress. Further it's not right to stop the load process and execute ALL events, which are currently in the queue. Because the main point is: we should use all cpu time for the loading process ... and not for the execution of e.g. repaint events. On the other side you should be aware, that the progress implementation uses Reschedule() for one purpose only: We need a relaiable office during loading/saving(!) of large documents ... so you can work with other documents inbetween. You cant speed up the pure loading time of one special calc document ... without affecting this balanced feature. What can we do? If you wish to speed up loading of calc documents ... please try to find the problem inside the calc module. If they call setValue() for every cell ... find a way how it can be made more intelligent there .-) BTW: I know that the calc module has implemented it's own time-check mechanism using system calls and further the have no well known range for it's used progress ... I've tried to fix that within the same cws, where I've fixed my progress implementation ... but there was not enough time doing so before Beta. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User as changed the following: What|Old value |New value Status|NEW |RESOLVED Resolution| |INVALID --- Additional comments from [EMAIL PROTECTED] Tue Feb 8 22:31:28 -0800 2005 --- AS-mmeeks: Please check out the new statusindicatorfactory.cxx file and test it again. May be it still fixes your problem. If not - please try to find the problem inside calc. I will try to find some time to verify, check in and test your patch then. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User mmeeks changed the following: What|Old value |New value Attachment is patch| |Created an attachment | |(id=22306) don't do so | |many reschedules --- Additional comments from [EMAIL PROTECTED] Mon Feb 7 08:22:02 -0800 2005 --- Created an attachment (id=22306) don't do so many reschedules - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=42221
User mmeeks changed the following:
What|Old value |New value
Issue type|DEFECT|PATCH
Target milestone|--- |OOo 2.0
--- Additional comments from [EMAIL PROTECTED] Mon Feb 7 08:25:11 -0800
2005 ---
The 2nd patch:
--- vcl/unx/gtk/app/gtkdata.cxx 21 Jan 2005 13:36:06 - 1.14
+++ vcl/unx/gtk/app/gtkdata.cxx 7 Feb 2005 16:24:47 -
@@ -699,7 +699,12 @@ void GtkXLib::Yield( BOOL bWait )
YieldMutexReleaser aReleaser;
if( bDispatchThread )
-g_main_context_iteration( NULL, bWait );
+ {
+ if( !bWait )
+ while( g_main_context_iteration( NULL, bWait )
);
+ else
+ g_main_context_iteration( NULL, TRUE );
+ }
else
osl_waitCondition( m_aDispatchCondition, NULL );
}
-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
[gsl-issues] [Issue 42221] interactivity yield on load ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=42221 User pl changed the following: What|Old value |New value Assigned to|pl|as --- Additional comments from [EMAIL PROTECTED] Mon Feb 7 08:35:16 -0800 2005 --- pl-as: please have a look at the statusbar patch (that's yours, isn't it ?) As for the Yield patch: it's nice that it will speed up calc loading; but i'm concerned what will break if you simply overthrow our current event model. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
