Skip to content

C
core
Kaydet (Commit) 84805862 authored tarafından Noel Grandin's avatar Noel Grandin
Dosyalara gözat
Seçenekler

use vcl::Timer in StatusIndicatorFactory 

does not need a separate thread

Change-Id: I47bf2b255a331f4ec3ea24ad3a5d4c3ca398557e
Reviewed-on: https://gerrit.libreoffice.org/72901
Tested-by: Jenkins
Reviewed-by: 's avatarNoel Grandin <noel.grandin@collabora.co.uk>
üst dfbce2a5
Show whitespace changes
Inline Side-by-side
Showing with 18 additions and 132 deletions
framework/Library_fwk.mk
Dosyayı görüntüle @ 84805862
......@@ -88,7 +88,6 @@ $(eval $(call gb_Library_add_exception_objects,fwk,\
framework/source/helper/uiconfigelementwrapperbase \
framework/source/helper/uielementwrapperbase \
framework/source/helper/vclstatusindicator \
framework/source/helper/wakeupthread \
framework/source/interaction/quietinteraction \
framework/source/jobs/job \
framework/source/jobs/jobdata \
......
framework/inc/helper/statusindicatorfactory.hxx
Dosyayı görüntüle @ 84805862
......@@ -25,7 +25,6 @@
#include <vector>
// include files of own module
#include <helper/wakeupthread.hxx>
#include <general.h>
// include uno interfaces
......@@ -45,7 +44,9 @@
#include <com/sun/star/uno/XComponentContext.hpp>
#include <cppuhelper/supportsservice.hxx>
#include <cppuhelper/weakref.hxx>
#include <vcl/status.hxx>
#include <vcl/timer.hxx>
#include <cppuhelper/implbase.hxx>
#include <osl/thread.hxx>
......@@ -165,7 +166,7 @@ class StatusIndicatorFactory : public ::cppu::WeakImplHelper<
/** Notify us if a fix time is over. We use it to implement an
intelligent "Reschedule" ... */
rtl::Reference<WakeUpThread> m_pWakeUp;
boost::optional<Timer> m_xWakeUpTimer;
/** Our WakeUpThread calls us in our interface method "XUpdatable::update().
There we set this member m_bAllowReschedule to sal_True. Next time if our impl_reschedule()
......@@ -182,6 +183,8 @@ class StatusIndicatorFactory : public ::cppu::WeakImplHelper<
/** prevent recursive calling of Application::Reschedule(). */
static sal_Int32 m_nInReschedule;
DECL_LINK( WakeupTimerHdl, Timer*, void );
// interface
public:
......
framework/inc/helper/wakeupthread.hxx deleted 100644 → 0
Dosyayı görüntüle @ dfbce2a5
/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
/*
* This file is part of the LibreOffice project.
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/.
*
* This file incorporates work covered by the following license notice:
*
* Licensed to the Apache Software Foundation (ASF) under one or more
* contributor license agreements. See the NOTICE file distributed
* with this work for additional information regarding copyright
* ownership. The ASF licenses this file to you under the Apache
* License, Version 2.0 (the "License"); you may not use this file
* except in compliance with the License. You may obtain a copy of
* the License at http://www.apache.org/licenses/LICENSE-2.0 .
*/
#ifndef INCLUDED_FRAMEWORK_INC_HELPER_WAKEUPTHREAD_HXX
#define INCLUDED_FRAMEWORK_INC_HELPER_WAKEUPTHREAD_HXX
#include <sal/config.h>
#include <com/sun/star/uno/Reference.hxx>
#include <cppuhelper/weakref.hxx>
#include <osl/conditn.hxx>
#include <osl/mutex.hxx>
#include <sal/types.h>
#include <salhelper/thread.hxx>
namespace com { namespace sun { namespace star { namespace util {
class XUpdatable;
} } } }
namespace framework{
class WakeUpThread: public salhelper::Thread {
css::uno::WeakReference<css::util::XUpdatable> updatable_;
osl::Condition condition_;
osl::Mutex mutex_;
bool terminate_;
void execute() override;
public:
WakeUpThread(css::uno::Reference<css::util::XUpdatable> const & updatable);
void stop();
};
}
#endif
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
framework/source/helper/statusindicatorfactory.cxx
Dosyayı görüntüle @ 84805862
......@@ -537,26 +537,27 @@ void StatusIndicatorFactory::impl_startWakeUpThread()
if (m_bDisableReschedule)
return;
if (!m_pWakeUp.is())
if (!m_xWakeUpTimer)
{
m_pWakeUp = new WakeUpThread(this);
m_pWakeUp->launch();
m_xWakeUpTimer = Timer();
m_xWakeUpTimer->SetInvokeHandler( LINK(this, StatusIndicatorFactory, WakeupTimerHdl) );
m_xWakeUpTimer->SetTimeout(25); // 25 msec
m_xWakeUpTimer->Start();
}
}
void StatusIndicatorFactory::impl_stopWakeUpThread()
{
rtl::Reference<WakeUpThread> wakeUp;
{
osl::MutexGuard g(m_mutex);
std::swap(wakeUp, m_pWakeUp);
}
if (wakeUp.is())
{
wakeUp->stop();
}
if (m_xWakeUpTimer)
m_xWakeUpTimer->Stop();
}
IMPL_LINK_NOARG(StatusIndicatorFactory, WakeupTimerHdl, Timer *, void)
{
update();
}
} // namespace framework
extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
......
framework/source/helper/wakeupthread.cxx deleted 100644 → 0
Dosyayı görüntüle @ dfbce2a5
/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
/*
* This file is part of the LibreOffice project.
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/.
*
* This file incorporates work covered by the following license notice:
*
* Licensed to the Apache Software Foundation (ASF) under one or more
* contributor license agreements. See the NOTICE file distributed
* with this work for additional information regarding copyright
* ownership. The ASF licenses this file to you under the Apache
* License, Version 2.0 (the "License"); you may not use this file
* except in compliance with the License. You may obtain a copy of
* the License at http://www.apache.org/licenses/LICENSE-2.0 .
*/
#include <sal/config.h>
#include <com/sun/star/uno/Reference.hxx>
#include <com/sun/star/util/XUpdatable.hpp>
#include <osl/mutex.hxx>
#include <osl/time.h>
#include <helper/wakeupthread.hxx>
void framework::WakeUpThread::execute() {
for (;;) {
TimeValue t{0, 25000000}; // 25 msec
condition_.wait(&t);
{
osl::MutexGuard g(mutex_);
if (terminate_) {
break;
}
}
css::uno::Reference<css::util::XUpdatable> up(updatable_);
if (up.is()) {
up->update();
}
}
}
framework::WakeUpThread::WakeUpThread(
css::uno::Reference<css::util::XUpdatable> const & updatable):
Thread("WakeUpThread"), updatable_(updatable), terminate_(false)
{}
void framework::WakeUpThread::stop() {
{
osl::MutexGuard g(mutex_);
terminate_ = true;
}
condition_.set();
join();
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment