[PATCH] Cleanup after zoom dialog ui conversion

2013-01-25 Thread Roth Robert (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1873

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/73/1873/1

Cleanup after zoom dialog ui conversion

Change-Id: I99cbb6e739af352091446da11d8150b24c2a6207
---
M cui/source/dialogs/zoom.cxx
D cui/source/dialogs/zoom.hrc
2 files changed, 0 insertions(+), 44 deletions(-)



diff --git a/cui/source/dialogs/zoom.cxx b/cui/source/dialogs/zoom.cxx
index 3121529..de5997f 100644
--- a/cui/source/dialogs/zoom.cxx
+++ b/cui/source/dialogs/zoom.cxx
@@ -27,7 +27,6 @@
 #define _SVX_ZOOM_CXX
 
 #include cuires.hrc
-#include zoom.hrc
 
 #include zoom.hxx
 #include sfx2/zoomitem.hxx
diff --git a/cui/source/dialogs/zoom.hrc b/cui/source/dialogs/zoom.hrc
deleted file mode 100644
index c2a5bbc..000
--- a/cui/source/dialogs/zoom.hrc
+++ /dev/null
@@ -1,43 +0,0 @@
-/*
- * 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 _SVX_ZOOM_HRC
-#define _SVX_ZOOM_HRC
-
-// defines --
-
-#define FL_ZOOM 14
-#define BTN_OPTIMAL 15
-#define BTN_WHOLE_PAGE  16
-#define BTN_PAGE_WIDTH  17
-#define BTN_100 18
-#define BTN_USER19
-#define ED_USER 20
-
-#define FL_VIEWLAYOUT   21
-#define BTN_AUTOMATIC   22
-#define BTN_SINGLE  23
-#define BTN_COLUMNS 24
-#define ED_COLUMNS  25
-#define CHK_BOOK26
-
-#define FL_BOTTOM   27
-#define BTN_ZOOM_OK 28
-#define BTN_ZOOM_CANCEL 29
-#define BTN_ZOOM_HELP   30
-
-#endif

-- 
To view, visit https://gerrit.libreoffice.org/1873
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I99cbb6e739af352091446da11d8150b24c2a6207
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Roth Robert robert.roth@gmail.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Licensing

2012-05-10 Thread Roth Robert
I might have forgotten to mention previously with my patches:
All of my past  future contributions to LibreOffice may be licensed under
the MPL/LGPLv3+ dual license.

Regards,
Robert
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] What is mediadescriptor.h

2010-12-14 Thread Roth Robert
I've been looking for something in the code, and I've found this
headerhttp://opengrok.go-oo.org/xref/libs-core/framework/inc/mediadescriptor.h,
mediadescriptor.h in libs-core/framework/inc, which declares an empty
namespace (all the lines are commented out), and as far as I've seen, this
header is not used anywhere. Can this be removed?

Regards,
  Robert
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] Extensions code cleanup

2010-11-25 Thread Roth Robert
Some more dead code and bogus comments removed from extensions. The license,
as always is LGPLv3+ / MPL.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] Fwd: Extensions code cleanup with patch

2010-11-25 Thread Roth Robert
-- Forwarded message --
From: Roth Robert robert.roth@gmail.com
Date: Thu, Nov 25, 2010 at 9:27 PM
Subject: Extensions code cleanup
To: libreoffice@lists.freedesktop.org


Some more dead code and bogus comments removed from extensions. The license,
as always is LGPLv3+ / MPL.
From 550313e5020f855fd1a733f8ac9aed269bd00de3 Mon Sep 17 00:00:00 2001
From: Robert Roth robert.roth@gmail.com
Date: Thu, 25 Nov 2010 21:22:04 +0200
Subject: [PATCH] Some more dead code removed from the extensions.

---
 extensions/source/bibliography/datman.cxx|   30 --
 extensions/source/bibliography/toolbar.cxx   |7 +
 extensions/source/bibliography/toolbar.hxx   |1 -
 extensions/source/dbpilots/controlwizard.cxx |   25 --
 extensions/source/dbpilots/groupboxwiz.cxx   |6 
 extensions/source/ole/ole2uno.cxx|   16 ---
 extensions/source/ole/unoconversionutilities.hxx |7 -
 extensions/source/ole/unotypewrapper.hxx |2 +-
 extensions/source/oooimprovement/invite_job.cxx  |9 --
 extensions/source/propctrlr/browserview.hxx  |2 -
 extensions/source/propctrlr/taborder.cxx |2 -
 extensions/source/scanner/sane.cxx   |1 -
 extensions/source/scanner/scanwin.cxx|   14 +-
 extensions/source/svg/svgwriter.cxx  |4 ---
 14 files changed, 9 insertions(+), 117 deletions(-)

diff --git a/extensions/source/bibliography/datman.cxx b/extensions/source/bibliography/datman.cxx
index 85fb0f9..3ad77ed 100644
--- a/extensions/source/bibliography/datman.cxx
+++ b/extensions/source/bibliography/datman.cxx
@@ -134,19 +134,12 @@ Reference XConnection  getConnection(const ::rtl::OUString _rURL)
 ::rtl::OUString sUser, sPwd;
 Reference XPropertySet   xDataSourceProps(xDataSource, UNO_QUERY);
 Reference XCompletedConnection  xComplConn(xDataSource, UNO_QUERY);
-/*		Reference XPropertySetInfo   xInfo = xDataSourceProps.is() ? xDataSourceProps-getPropertySetInfo() : Reference XPropertySetInfo  ();
-if (xInfo.is()  xInfo-hasPropertyByName(C2U(User)))
-xDataSourceProps-getPropertyValue(C2U(User)) = sUser;
-if (xInfo.is()  xInfo-hasPropertyByName(C2U(Password)))
-xDataSourceProps-getPropertyValue(C2U(Password))= sPwd;
-*/
 try
 {
 
 ReferenceXInterface xHdl = xMgr-createInstance(C2U(com.sun.star.task.InteractionHandler));
 Referencetask::XInteractionHandler xIHdl(xHdl, UNO_QUERY);
 xConn = xComplConn-connectWithCompletion(xIHdl);
-//			xConn = xDataSource-getConnection(sUser, sPwd);
 }
 catch(SQLException)
 {
@@ -582,7 +575,6 @@ class DBChangeDialog_Impl : public ModalDialog
 
 BibDataManager* pDatMan;
 
-//	DECL_LINK(EndDragHdl, HeaderBar*);
 DECL_LINK(DoubleClickHdl, SvTabListBox*);
 public:
 DBChangeDialog_Impl(Window* pParent, BibDataManager* pMan );
@@ -651,16 +643,6 @@ IMPL_LINK(DBChangeDialog_Impl, DoubleClickHdl, SvTabListBox*, /*pLB*/)
 return 0;
 }
 
-/*IMPL_LINK(DBChangeDialog_Impl, EndDragHdl, HeaderBar*, pHB)
-{
-long nTabs[3];
-nTabs[0] = 2;// Number of Tabs
-nTabs[1] = 0;
-nTabs[2] = pHB-GetItemSize( 1 );
-aSelectionLB.SetTabs( nTabs[0], MAP_PIXEL );
-return 0;
-};*/
-
 DBChangeDialog_Impl::~DBChangeDialog_Impl()
 {
 }
@@ -676,7 +658,6 @@ String	DBChangeDialog_Impl::GetCurrentURL()const
 return sRet;
 }
 
-// #100312# 
 // XDispatchProvider
 BibInterceptorHelper::BibInterceptorHelper( ::bib::BibBeamer* pBibBeamer, ::com::sun::star::uno::Reference ::com::sun::star::frame::XDispatch  xDispatch)
 {
@@ -762,7 +743,6 @@ void SAL_CALL BibInterceptorHelper::setMasterDispatchProvider( const ::com::sun:
 
 BibDataManager::BibDataManager()
 :BibDataManager_Base( GetMutex() )
-// #100312# --
 ,m_pInterceptorHelper( NULL )
 ,m_aLoadListeners(m_aMutex)
 ,pBibView( NULL )
@@ -791,7 +771,6 @@ BibDataManager::~BibDataManager()
 xConnection-dispose();
 m_xForm = NULL;
 }
-// #100312# 
 if( m_pInterceptorHelper )
 {
 m_pInterceptorHelper-ReleaseInterceptor();
@@ -910,9 +889,6 @@ Reference awt::XControlModel 	BibDataManager::updateGridModel(const Reference
 m_xGridModel = createGridModel( gGridName );
 
 Reference XNameContainer   xNameCont(xDbForm, UNO_QUERY);
-//			if (xNameCont-hasByName(sName))
-//xNameCont-removeByName(sName);
-//
 xNameCont-insertByName( sName, makeAny( m_xGridModel ) );
 }
 
@@ -1513,7 +1489,6 @@ Reference awt::XControlModel 	BibDataManager::loadControlModel(
 // (as an anologon to the XStatusListener semantics).
 //
 // But this would be way too risky for this last-day fix here.
-// 97140