ClickLightHdl doesn't need to be a link

and ClickLightHdl always takes a non-NULL argument, so NULL test is
pointless. Turn it into a normal method that takes a reference.
GetLightSource can be const. Nothing calls DoubleClickHdl, remove it.

svx/inc/svx/float3d.hxx

@@ -230,9 +230,7 @@ private:
     DECL_LINK( ClickColorHdl, PushButton * );
     DECL_LINK( SelectHdl, void * );
     DECL_LINK( ModifyHdl, void * );
-    DECL_LINK( ClickLightHdl, PushButton * );
-
-    DECL_LINK( DoubleClickHdl, void * );
+    void ClickLight(PushButton &rBtn);
 
     DECL_LINK( ChangeLightCallbackHdl, void * );
     DECL_LINK( ChangeSelectionCallbackHdl, void * );
@@ -244,7 +242,7 @@ private:
     SVX_DLLPRIVATE sal_uInt16           GetLightSource( const PushButton* pBtn = NULL );
     SVX_DLLPRIVATE ColorLB*     GetLbByButton( const PushButton* pBtn = NULL );
 
-    SVX_DLLPRIVATE bool         GetUILightState( ImageButton& aBtn ) const;
+    SVX_DLLPRIVATE bool         GetUILightState( const ImageButton& rBtn ) const;
     SVX_DLLPRIVATE void         SetUILightState( ImageButton& aBtn, bool bState );
 
 protected:

svx/source/engine3d/float3d.cxx

@@ -483,9 +483,9 @@ void Svx3DWin::Reset()
     aCtlLightPreview.GetSvx3DLightControl().SelectLight(0);
 }
 
-bool Svx3DWin::GetUILightState( ImageButton& aBtn ) const
+bool Svx3DWin::GetUILightState( const ImageButton& rBtn ) const
 {
-    return (aBtn.GetModeImage() == aImgLightOn);
+    return (rBtn.GetModeImage() == aImgLightOn);
 }
 
 void Svx3DWin::SetUILightState( ImageButton& aBtn, bool bState )
@@ -2718,7 +2718,7 @@ IMPL_LINK( Svx3DWin, ClickHdl, PushButton *, pBtn )
             aBtnLightColor.Enable( bEnable );
             pLb->Enable( bEnable );
 
-            ClickLightHdl( pBtn );
+            ClickLight(*pBtn);
             bUpdatePreview = sal_True;
         }
         // Textures
@@ -2950,16 +2950,13 @@ IMPL_LINK( Svx3DWin, ModifyHdl, void*, pField )
 
 // -----------------------------------------------------------------------
 
-IMPL_LINK( Svx3DWin, ClickLightHdl, PushButton*, pBtn )
+void Svx3DWin::ClickLight(PushButton& rBtn)
 {
-
-    if( pBtn )
-    {
-        sal_uInt16 nLightSource = GetLightSource( pBtn );
-        ColorLB* pLb = GetLbByButton( pBtn );
+    sal_uInt16 nLightSource = GetLightSource( &rBtn );
+    ColorLB* pLb = GetLbByButton( &rBtn );
     Color aColor( pLb->GetSelectEntryColor() );
     SfxItemSet aLightItemSet(aCtlLightPreview.GetSvx3DLightControl().Get3DAttributes());
-        const bool bOnOff(GetUILightState( *(ImageButton*)pBtn ));
+    const bool bOnOff(GetUILightState( (const ImageButton&)rBtn ));
 
     switch(nLightSource)
     {
@@ -2977,18 +2974,6 @@ IMPL_LINK( Svx3DWin, ClickLightHdl, PushButton*, pBtn )
     aCtlLightPreview.GetSvx3DLightControl().Set3DAttributes(aLightItemSet);
     aCtlLightPreview.GetSvx3DLightControl().SelectLight(nLightSource);
     aCtlLightPreview.CheckSelection();
-    }
-    return( 0L );
-}
-
-
-// -----------------------------------------------------------------------
-IMPL_LINK_NOARG(Svx3DWin, DoubleClickHdl)
-{
-    // and assign
-    ClickAssignHdl( NULL );
-
-    return( 0L );
 }
 
 // -----------------------------------------------------------------------

unusedcode.easy

@@ -201,8 +201,6 @@ SvXMLTokenMap_Impl::Insert(SvXMLTokenMapEntry_Impl* const*, unsigned short)
 SvXMLTokenMap_Impl::Insert(SvXMLTokenMap_Impl const*, unsigned short, unsigned short)
 SvXMLTokenMap_Impl::Remove(SvXMLTokenMapEntry_Impl* const&, unsigned short)
 SvXMLTokenMap_Impl::Remove(unsigned short, unsigned short)
-Svx3DWin::LinkStubClickLightHdl(void*, void*)
-Svx3DWin::LinkStubDoubleClickHdl(void*, void*)
 SvxAreaTabPage::LinkStubClickBitmapHdl_Impl(void*, void*)
 SvxAreaTabPage::LinkStubClickColorHdl_Impl(void*, void*)
 SvxAreaTabPage::LinkStubClickGradientHdl_Impl(void*, void*)