Bug 97102 - Re-work copy/paste 'orrible extensions dialog code...
Summary: Re-work copy/paste 'orrible extensions dialog code...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Extensions (show other bugs)
Version:
(earliest affected)
5.1.0.1 rc
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: difficultyInteresting, easyHack, skillCpp, topicCleanup
Depends on:
Blocks: Dialog
  Show dependency treegraph
 
Reported: 2016-01-13 14:44 UTC by Michael Meeks
Modified: 2017-07-26 19:19 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Meeks 2016-01-13 14:44:00 UTC
There is a chunk of code in:

desktop/source/deployment/gui/dp_gui_dialog2.cxx

with some pretty bad practice on display. There is a large chunk of cut-and-paste coding between:

IMPL_LINK_NOARG_TYPED(UpdateRequiredDialog, TimeOutHdl, Idle *, void)
and
IMPL_LINK_NOARG_TYPED(ExtMgrDialog, TimeOutHdl, Idle *, void)

And that code itself should -not- be in an idle handler that runs all the time. Instead we should add an idle task whenever anything changes.

The reason the code is done like this is to avoid needing to take the SolarMutex in the implementation that updates the progress etc. which is useful in several cases for UNO components etc.

Anyhow - we should create a new (inside VCL) "ProgressBarAsync" class and encapsulate this functionality into VCL.

The methods - such as updating the progress %age, and the text etc. should take effect only asynchronously in an idle handler -but- the idle handler should only be triggered when some state actually changes =) a new %age, new text etc.

Then we should clean the cut/paste coding out of the above module.

Thanks !
Comment 1 Robinson Tryon (qubit) 2016-02-18 14:52:28 UTC Comment hidden (obsolete)
Comment 2 Xisco Faulí 2017-07-26 19:19:58 UTC
Hi Michael,
it seems the code pointers are no longer valid.
Could you please update them?