Make ImplSVData private to vcl

Change-Id: I1df6ba8a269be3e165ff5948af1dd12e0bdc6f7d

include/vcl/scheduler.hxx

@@ -23,7 +23,6 @@
 #include <vcl/dllapi.h>
 
 class Scheduler;
-struct ImplSVData;
 struct ImplSchedulerData;
 
 enum class SchedulerPriority {
@@ -39,9 +38,6 @@ enum class SchedulerPriority {
 
 class VCL_DLLPUBLIC Scheduler
 {
-private:
-    static void InitSystemTimer(ImplSVData* pSVData);
-
 protected:
     ImplSchedulerData*  mpSchedulerData;    /// Pointer to element in scheduler list
     const sal_Char     *mpDebugName;        /// Useful for debugging
@@ -50,7 +46,6 @@ protected:
 
     // These should be constexpr static, when supported.
     static const sal_uInt64 ImmediateTimeoutMs = 1;
-    static const sal_uInt64 MaximumTimeoutMs = 1000 * 60; // 1 minute
 
     static void ImplStartTimer(sal_uInt64 nMS, bool bForce = false);
 

include/vcl/svapp.hxx

@@ -1450,9 +1450,6 @@ public:
     static void setDeInitHook(Link<LinkParamNone*,void> const & hook);
 
 private:
-
-    static void InitSettings(ImplSVData* pSVData);
-
     DECL_STATIC_LINK_TYPED( Application, PostEventHandler, void*, void );
 };
 

vcl/source/app/scheduler.cxx

@@ -24,6 +24,11 @@
 #include <svdata.hxx>
 #include <salinst.hxx>
 
+namespace {
+const sal_uInt64 MaximumTimeoutMs = 1000 * 60; // 1 minute
+void InitSystemTimer(ImplSVData* pSVData);
+}
+
 void ImplSchedulerData::Invoke()
 {
     if (mbDelete || mbInScheduler )
@@ -135,21 +140,25 @@ void Scheduler::ImplStartTimer(sal_uInt64 nMS, bool bForce)
     }
 }
 
+namespace {
+
 /**
 * Initialize the platform specific timer on which all the
 * platform independent timers are built
 */
-void Scheduler::InitSystemTimer(ImplSVData* pSVData)
+void InitSystemTimer(ImplSVData* pSVData)
 {
     assert(pSVData != nullptr);
     if (!pSVData->mpSalTimer)
     {
         pSVData->mnTimerPeriod = MaximumTimeoutMs;
         pSVData->mpSalTimer = pSVData->mpDefInst->CreateSalTimer();
-        pSVData->mpSalTimer->SetCallback(CallbackTaskScheduling);
+        pSVData->mpSalTimer->SetCallback(Scheduler::CallbackTaskScheduling);
     }
 }
 
+}
+
 void Scheduler::CallbackTaskScheduling(bool ignore)
 {
     // this function is for the saltimer callback

vcl/source/app/svapp.cxx

@@ -80,6 +80,10 @@
 using namespace ::com::sun::star;
 using namespace ::com::sun::star::uno;
 
+namespace {
+void InitSettings(ImplSVData* pSVData);
+}
+
 // keycodes handled internally by VCL
 class ImplReservedKey
 {
@@ -769,7 +773,9 @@ const AllSettings& Application::GetSettings()
     return *(pSVData->maAppData.mpSettings);
 }
 
-void Application::InitSettings(ImplSVData* pSVData)
+namespace {
+
+void InitSettings(ImplSVData* pSVData)
 {
     assert(!pSVData->maAppData.mpSettings && "initialization should not happen twice!");
 
@@ -781,6 +787,8 @@ void Application::InitSettings(ImplSVData* pSVData)
     }
 }
 
+}
+
 void Application::NotifyAllWindows( DataChangedEvent& rDCEvt )
 {
     ImplSVData* pSVData = ImplGetSVData();