[Libreoffice-commits] .: 2 commits - basctl/Library_basctl.mk basctl/source
August Sodora
augsod at kemper.freedesktop.org
Sun Dec 4 19:02:11 PST 2011
basctl/Library_basctl.mk | 1
basctl/source/basicide/baside2.hxx | 1
basctl/source/basicide/baside2b.cxx | 2
basctl/source/basicide/basidesh.cxx | 4
basctl/source/basicide/bastypes.cxx | 125 -----------------------
basctl/source/basicide/breakpoint.cxx | 179 ++++++++++++++++++++++++++++++++++
basctl/source/basicide/breakpoint.hxx | 79 +++++++++++++++
basctl/source/basicide/brkdlg.cxx | 15 +-
basctl/source/inc/bastypes.hxx | 73 +------------
9 files changed, 279 insertions(+), 200 deletions(-)
New commits:
commit f81a7438409ee9cede09b0395d8af0943220a4ea
Author: August Sodora <augsod at gmail.com>
Date: Sun Dec 4 22:01:35 2011 -0500
Refactor BreakPoint
diff --git a/basctl/Library_basctl.mk b/basctl/Library_basctl.mk
index 45403ec..438c8c3 100644
--- a/basctl/Library_basctl.mk
+++ b/basctl/Library_basctl.mk
@@ -85,6 +85,7 @@ $(eval $(call gb_Library_add_exception_objects,basctl,\
basctl/source/basicide/bastype2 \
basctl/source/basicide/bastype3 \
basctl/source/basicide/bastypes \
+ basctl/source/basicide/breakpoint \
basctl/source/basicide/brkdlg \
basctl/source/basicide/doceventnotifier \
basctl/source/basicide/docsignature \
diff --git a/basctl/source/basicide/baside2.hxx b/basctl/source/basicide/baside2.hxx
index 119aa60..e9378ab 100644
--- a/basctl/source/basicide/baside2.hxx
+++ b/basctl/source/basicide/baside2.hxx
@@ -51,6 +51,7 @@ class SvxSearchItem;
#include <sfx2/progress.hxx>
#include <svtools/syntaxhighlight.hxx>
+#include "breakpoint.hxx"
#include "linenumberwindow.hxx"
DBG_NAMEEX( ModulWindow )
diff --git a/basctl/source/basicide/baside2b.cxx b/basctl/source/basicide/baside2b.cxx
index 4ad4eea..dfef385 100644
--- a/basctl/source/basicide/baside2b.cxx
+++ b/basctl/source/basicide/baside2b.cxx
@@ -26,14 +26,12 @@
*
************************************************************************/
-
#define _BASIC_TEXTPORTIONS
#include "helpid.hrc"
#include "baside2.hrc"
#include "baside2.hxx"
-#include "basobj.hxx"
#include "brkdlg.hxx"
#include "iderdll.hxx"
#include "iderdll2.hxx"
diff --git a/basctl/source/basicide/bastypes.cxx b/basctl/source/basicide/bastypes.cxx
index 2ced841..21dfce9 100644
--- a/basctl/source/basicide/bastypes.cxx
+++ b/basctl/source/basicide/bastypes.cxx
@@ -252,127 +252,6 @@ Window* IDEBaseWindow::GetLayoutWindow()
return NULL;
}
-BreakPointList::BreakPointList()
-{}
-
-BreakPointList::BreakPointList(BreakPointList const & rList)
-{
- for (size_t i = 0; i < rList.size(); ++i)
- maBreakPoints.push_back( new BreakPoint(*rList.at( i ) ) );
-}
-
-BreakPointList::~BreakPointList()
-{
- reset();
-}
-
-void BreakPointList::reset()
-{
- for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i )
- delete maBreakPoints[ i ];
- maBreakPoints.clear();
-}
-
-void BreakPointList::transfer(BreakPointList & rList)
-{
- reset();
- for (size_t i = 0; i < rList.size(); ++i)
- maBreakPoints.push_back( rList.at( i ) );
- rList.clear();
-}
-
-void BreakPointList::InsertSorted( BreakPoint* pNewBrk )
-{
- for ( vector< BreakPoint* >::iterator i = maBreakPoints.begin(); i < maBreakPoints.end(); ++i )
- {
- if ( pNewBrk->nLine <= (*i)->nLine )
- {
- DBG_ASSERT( ( (*i)->nLine != pNewBrk->nLine ) || pNewBrk->bTemp, "BreakPoint existiert schon!" );
- maBreakPoints.insert( i, pNewBrk );
- return;
- }
- }
- // no insert position found => LIST_APPEND
- maBreakPoints.push_back( pNewBrk );
-}
-
-void BreakPointList::SetBreakPointsInBasic( SbModule* pModule )
-{
- pModule->ClearAllBP();
-
- for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i )
- {
- BreakPoint* pBrk = maBreakPoints[ i ];
- if ( pBrk->bEnabled )
- pModule->SetBP( (sal_uInt16)pBrk->nLine );
- }
-}
-
-BreakPoint* BreakPointList::FindBreakPoint( size_t nLine )
-{
- for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i )
- {
- BreakPoint* pBrk = maBreakPoints[ i ];
- if ( pBrk->nLine == nLine )
- return pBrk;
- }
- return NULL;
-}
-
-void BreakPointList::AdjustBreakPoints( size_t nLine, bool bInserted )
-{
- for ( size_t i = 0; i < maBreakPoints.size(); )
- {
- BreakPoint* pBrk = maBreakPoints[ i ];
- bool bDelBrk = false;
- if ( pBrk->nLine == nLine )
- {
- if ( bInserted )
- pBrk->nLine++;
- else
- bDelBrk = true;
- }
- else if ( pBrk->nLine > nLine )
- {
- if ( bInserted )
- pBrk->nLine++;
- else
- pBrk->nLine--;
- }
-
- if ( bDelBrk )
- {
- delete remove( pBrk );
- }
- else
- {
- ++i;
- }
- }
-}
-
-void BreakPointList::ResetHitCount()
-{
- for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i )
- {
- BreakPoint* pBrk = maBreakPoints[ i ];
- pBrk->nHitCount = 0;
- }
-}
-
-BreakPoint* BreakPointList::remove( BreakPoint* ptr )
-{
- for ( vector< BreakPoint* >::iterator i = maBreakPoints.begin(); i < maBreakPoints.end(); ++i )
- {
- if ( ptr == *i )
- {
- maBreakPoints.erase( i );
- return ptr;
- }
- }
- return NULL;
-}
-
void IDEBaseWindow::Deactivating()
{
}
diff --git a/basctl/source/basicide/breakpoint.cxx b/basctl/source/basicide/breakpoint.cxx
new file mode 100644
index 0000000..b763174
--- /dev/null
+++ b/basctl/source/basicide/breakpoint.cxx
@@ -0,0 +1,179 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*************************************************************************
+ *
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * Copyright 2000, 2010 Oracle and/or its affiliates.
+ *
+ * OpenOffice.org - a multi-platform office productivity suite
+ *
+ * This file is part of OpenOffice.org.
+ *
+ * OpenOffice.org is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License version 3
+ * only, as published by the Free Software Foundation.
+ *
+ * OpenOffice.org is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Lesser General Public License version 3 for more details
+ * (a copy is included in the LICENSE file that accompanied this code).
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * version 3 along with OpenOffice.org. If not, see
+ * <http://www.openoffice.org/license.html>
+ * for a copy of the LGPLv3 License.
+ *
+ ************************************************************************/
+
+#include "breakpoint.hxx"
+
+#include <basic/sbmod.hxx>
+#include <tools/debug.hxx>
+
+BreakPointList::BreakPointList()
+{}
+
+BreakPointList::BreakPointList(BreakPointList const & rList)
+{
+ for (size_t i = 0; i < rList.size(); ++i)
+ maBreakPoints.push_back( new BreakPoint(*rList.at( i ) ) );
+}
+
+BreakPointList::~BreakPointList()
+{
+ reset();
+}
+
+void BreakPointList::reset()
+{
+ for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i )
+ delete maBreakPoints[ i ];
+ maBreakPoints.clear();
+}
+
+void BreakPointList::transfer(BreakPointList & rList)
+{
+ reset();
+ for (size_t i = 0; i < rList.size(); ++i)
+ maBreakPoints.push_back( rList.at( i ) );
+ rList.clear();
+}
+
+void BreakPointList::InsertSorted(BreakPoint* pNewBrk)
+{
+ for ( ::std::vector< BreakPoint* >::iterator i = maBreakPoints.begin(); i < maBreakPoints.end(); ++i )
+ {
+ if ( pNewBrk->nLine <= (*i)->nLine )
+ {
+ DBG_ASSERT( ( (*i)->nLine != pNewBrk->nLine ) || pNewBrk->bTemp, "BreakPoint existiert schon!" );
+ maBreakPoints.insert( i, pNewBrk );
+ return;
+ }
+ }
+ // no insert position found => LIST_APPEND
+ maBreakPoints.push_back( pNewBrk );
+}
+
+void BreakPointList::SetBreakPointsInBasic(SbModule* pModule)
+{
+ pModule->ClearAllBP();
+
+ for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i )
+ {
+ BreakPoint* pBrk = maBreakPoints[ i ];
+ if ( pBrk->bEnabled )
+ pModule->SetBP( (sal_uInt16)pBrk->nLine );
+ }
+}
+
+BreakPoint* BreakPointList::FindBreakPoint(size_t nLine)
+{
+ for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i )
+ {
+ BreakPoint* pBrk = maBreakPoints[ i ];
+ if ( pBrk->nLine == nLine )
+ return pBrk;
+ }
+ return NULL;
+}
+
+void BreakPointList::AdjustBreakPoints(size_t nLine, bool bInserted)
+{
+ for ( size_t i = 0; i < maBreakPoints.size(); )
+ {
+ BreakPoint* pBrk = maBreakPoints[ i ];
+ bool bDelBrk = false;
+ if ( pBrk->nLine == nLine )
+ {
+ if ( bInserted )
+ pBrk->nLine++;
+ else
+ bDelBrk = true;
+ }
+ else if ( pBrk->nLine > nLine )
+ {
+ if ( bInserted )
+ pBrk->nLine++;
+ else
+ pBrk->nLine--;
+ }
+
+ if ( bDelBrk )
+ {
+ delete remove( pBrk );
+ }
+ else
+ {
+ ++i;
+ }
+ }
+}
+
+void BreakPointList::ResetHitCount()
+{
+ for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i )
+ {
+ BreakPoint* pBrk = maBreakPoints[ i ];
+ pBrk->nHitCount = 0;
+ }
+}
+
+BreakPoint* BreakPointList::remove(BreakPoint* ptr)
+{
+ for ( ::std::vector< BreakPoint* >::iterator i = maBreakPoints.begin(); i < maBreakPoints.end(); ++i )
+ {
+ if ( ptr == *i )
+ {
+ maBreakPoints.erase( i );
+ return ptr;
+ }
+ }
+ return NULL;
+}
+
+size_t BreakPointList::size() const
+{
+ return maBreakPoints.size();
+}
+
+BreakPoint* BreakPointList::at(size_t i)
+{
+ return i < maBreakPoints.size() ? maBreakPoints[ i ] : NULL;
+}
+
+const BreakPoint* BreakPointList::at(size_t i) const
+{
+ return i < maBreakPoints.size() ? maBreakPoints[ i ] : NULL;
+}
+
+void BreakPointList::push_back(BreakPoint* item)
+{
+ maBreakPoints.push_back( item );
+}
+
+void BreakPointList::clear()
+{
+ maBreakPoints.clear();
+}
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/basctl/source/basicide/breakpoint.hxx b/basctl/source/basicide/breakpoint.hxx
new file mode 100644
index 0000000..16cdfa3
--- /dev/null
+++ b/basctl/source/basicide/breakpoint.hxx
@@ -0,0 +1,79 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*************************************************************************
+ *
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * Copyright 2000, 2010 Oracle and/or its affiliates.
+ *
+ * OpenOffice.org - a multi-platform office productivity suite
+ *
+ * This file is part of OpenOffice.org.
+ *
+ * OpenOffice.org is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License version 3
+ * only, as published by the Free Software Foundation.
+ *
+ * OpenOffice.org is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Lesser General Public License version 3 for more details
+ * (a copy is included in the LICENSE file that accompanied this code).
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * version 3 along with OpenOffice.org. If not, see
+ * <http://www.openoffice.org/license.html>
+ * for a copy of the LGPLv3 License.
+ *
+ ************************************************************************/
+#ifndef _BREAKPOINT_HXX
+#define _BREAKPOINT_HXX
+
+#include <vector>
+
+class SbModule;
+
+struct BreakPoint
+{
+ bool bEnabled;
+ bool bTemp;
+ size_t nLine;
+ size_t nStopAfter;
+ size_t nHitCount;
+
+ BreakPoint(size_t nL) { nLine = nL; nStopAfter = 0; nHitCount = 0; bEnabled = true; bTemp = false; }
+};
+
+class BreakPointList
+{
+private:
+ void operator =(BreakPointList); // not implemented
+ ::std::vector<BreakPoint*> maBreakPoints;
+
+public:
+ BreakPointList();
+
+ BreakPointList(BreakPointList const & rList);
+
+ ~BreakPointList();
+
+ void reset();
+
+ void transfer(BreakPointList & rList);
+
+ void InsertSorted(BreakPoint* pBrk);
+ BreakPoint* FindBreakPoint(size_t nLine);
+ void AdjustBreakPoints(size_t nLine, bool bInserted);
+ void SetBreakPointsInBasic(SbModule* pModule);
+ void ResetHitCount();
+
+ size_t size() const;
+ BreakPoint* at(size_t i);
+ const BreakPoint* at(size_t i) const;
+ void push_back(BreakPoint* item);
+ void clear();
+ BreakPoint* remove(BreakPoint* ptr);
+};
+
+#endif // _BREAKPOINT_HXX
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/basctl/source/basicide/brkdlg.cxx b/basctl/source/basicide/brkdlg.cxx
index 5dd4b3a..3365e13 100644
--- a/basctl/source/basicide/brkdlg.cxx
+++ b/basctl/source/basicide/brkdlg.cxx
@@ -26,17 +26,16 @@
*
************************************************************************/
-
-#include <limits>
-
#include <vcl/sound.hxx>
#define _SVX_NOIDERESIDS
-#include <brkdlg.hxx>
-#include <brkdlg.hrc>
-#include <basidesh.hxx>
-#include <basidesh.hrc>
-#include <iderdll.hxx>
+#include "breakpoint.hxx"
+#include "brkdlg.hxx"
+#include "brkdlg.hrc"
+#include "basidesh.hxx"
+#include "basidesh.hrc"
+#include "iderdll.hxx"
+
#include <comphelper/string.hxx>
#include <sfx2/dispatch.hxx>
#include <sfx2/viewfrm.hxx>
diff --git a/basctl/source/inc/bastypes.hxx b/basctl/source/inc/bastypes.hxx
index e8c89f0..fb24a2b 100644
--- a/basctl/source/inc/bastypes.hxx
+++ b/basctl/source/inc/bastypes.hxx
@@ -65,17 +65,6 @@ struct BasicStatus
nBasicFlags(0) { }
};
-struct BreakPoint
-{
- bool bEnabled;
- bool bTemp;
- size_t nLine;
- size_t nStopAfter;
- size_t nHitCount;
-
- BreakPoint( size_t nL ) { nLine = nL; nStopAfter = 0; nHitCount = 0; bEnabled = true; bTemp = false; }
-};
-
class BasicDockingWindow : public DockingWindow
{
Rectangle aFloatingPosAndSize;
@@ -91,52 +80,6 @@ public:
BasicDockingWindow( Window* pParent );
};
-class BreakPointList
-{
-private:
- void operator =(BreakPointList); // not implemented
- ::std::vector< BreakPoint* > maBreakPoints;
-
-public:
- BreakPointList();
-
- BreakPointList(BreakPointList const & rList);
-
- ~BreakPointList();
-
- void reset();
-
- void transfer(BreakPointList & rList);
-
- void InsertSorted( BreakPoint* pBrk );
- BreakPoint* FindBreakPoint( size_t nLine );
- void AdjustBreakPoints( size_t nLine, bool bInserted );
- void SetBreakPointsInBasic( SbModule* pModule );
- void ResetHitCount();
-
- size_t size() const
- {
- return maBreakPoints.size();
- }
- BreakPoint* at( size_t i )
- {
- return i < maBreakPoints.size() ? maBreakPoints[ i ] : NULL;
- }
- const BreakPoint* at( size_t i ) const
- {
- return i < maBreakPoints.size() ? maBreakPoints[ i ] : NULL;
- }
- void push_back( BreakPoint* item )
- {
- maBreakPoints.push_back( item );
- }
- void clear()
- {
- maBreakPoints.clear();
- }
- BreakPoint* remove( BreakPoint* ptr );
-};
-
// helper class for sorting TabBar
class TabBarSortHelper
{
commit e7b6c4ac5fc38a05571201b92b81b163348602b5
Author: August Sodora <augsod at gmail.com>
Date: Sun Dec 4 20:17:29 2011 -0500
String->OUString
diff --git a/basctl/source/basicide/basidesh.cxx b/basctl/source/basicide/basidesh.cxx
index 65c92d6..e99224c 100644
--- a/basctl/source/basicide/basidesh.cxx
+++ b/basctl/source/basicide/basidesh.cxx
@@ -799,7 +799,7 @@ void BasicIDEShell::UpdateWindows()
for ( sal_Int32 j = 0 ; j < nModCount ; j++ )
{
- String aModName = pModNames[ j ];
+ ::rtl::OUString aModName = pModNames[ j ];
ModulWindow* pWin = FindBasWin( *doc, aLibName, aModName, sal_False );
if ( !pWin )
pWin = CreateBasWin( *doc, aLibName, aModName );
@@ -828,7 +828,7 @@ void BasicIDEShell::UpdateWindows()
for ( sal_Int32 j = 0 ; j < nDlgCount ; j++ )
{
- String aDlgName = pDlgNames[ j ];
+ ::rtl::OUString aDlgName = pDlgNames[ j ];
// this find only looks for non-suspended windows;
// suspended windows are handled in CreateDlgWin
DialogWindow* pWin = FindDlgWin( *doc, aLibName, aDlgName, sal_False );
diff --git a/basctl/source/basicide/bastypes.cxx b/basctl/source/basicide/bastypes.cxx
index f7dc4e5..2ced841 100644
--- a/basctl/source/basicide/bastypes.cxx
+++ b/basctl/source/basicide/bastypes.cxx
@@ -760,7 +760,7 @@ sal_uLong CalcLineCount( SvStream& rStream )
return nCRs;
}
-LibInfoKey::LibInfoKey( const ScriptDocument& rDocument, const String& rLibName )
+LibInfoKey::LibInfoKey( const ScriptDocument& rDocument, const ::rtl::OUString& rLibName )
:m_aDocument( rDocument )
,m_aLibName( rLibName )
{
@@ -791,7 +791,7 @@ bool LibInfoKey::operator==( const LibInfoKey& rKey ) const
return bRet;
}
-LibInfoItem::LibInfoItem( const ScriptDocument& rDocument, const String& rLibName, const String& rCurrentName, sal_uInt16 nCurrentType )
+LibInfoItem::LibInfoItem( const ScriptDocument& rDocument, const ::rtl::OUString& rLibName, const ::rtl::OUString& rCurrentName, sal_uInt16 nCurrentType )
:m_aDocument( rDocument )
,m_aLibName( rLibName )
,m_aCurrentName( rCurrentName )
diff --git a/basctl/source/inc/bastypes.hxx b/basctl/source/inc/bastypes.hxx
index 8684fe5..e8c89f0 100644
--- a/basctl/source/inc/bastypes.hxx
+++ b/basctl/source/inc/bastypes.hxx
@@ -262,10 +262,10 @@ class LibInfoKey
{
private:
ScriptDocument m_aDocument;
- String m_aLibName;
+ ::rtl::OUString m_aLibName;
public:
- LibInfoKey( const ScriptDocument& rDocument, const String& rLibName );
+ LibInfoKey( const ScriptDocument& rDocument, const ::rtl::OUString& rLibName );
~LibInfoKey();
LibInfoKey( const LibInfoKey& rKey );
@@ -275,19 +275,19 @@ public:
const ScriptDocument&
GetDocument() const { return m_aDocument; }
- const String& GetLibName() const { return m_aLibName; }
+ const ::rtl::OUString& GetLibName() const { return m_aLibName; }
};
class LibInfoItem
{
private:
ScriptDocument m_aDocument;
- String m_aLibName;
- String m_aCurrentName;
+ ::rtl::OUString m_aLibName;
+ ::rtl::OUString m_aCurrentName;
sal_uInt16 m_nCurrentType;
public:
- LibInfoItem( const ScriptDocument& rDocument, const String& rLibName, const String& rCurrentName, sal_uInt16 nCurrentType );
+ LibInfoItem( const ScriptDocument& rDocument, const ::rtl::OUString& rLibName, const ::rtl::OUString& rCurrentName, sal_uInt16 nCurrentType );
~LibInfoItem();
LibInfoItem( const LibInfoItem& rItem );
@@ -295,8 +295,8 @@ public:
const ScriptDocument&
GetDocument() const { return m_aDocument; }
- const String& GetLibName() const { return m_aLibName; }
- const String& GetCurrentName() const { return m_aCurrentName; }
+ const ::rtl::OUString& GetLibName() const { return m_aLibName; }
+ const ::rtl::OUString& GetCurrentName() const { return m_aCurrentName; }
sal_uInt16 GetCurrentType() const { return m_nCurrentType; }
};
More information about the Libreoffice-commits
mailing list