include/editeng/ESelection.hxx |  120 +++++++++++++++++++++++++++++++++++++++++
 include/editeng/editdata.hxx   |   90 ------------------------------
 2 files changed, 121 insertions(+), 89 deletions(-)

New commits:
commit 29097009b1f8c0dcb050367d3f2acfcaf2074a56
Author:     Tomaž Vajngerl <tomaz.vajng...@collabora.co.uk>
AuthorDate: Mon Dec 4 10:12:54 2023 +0900
Commit:     Tomaž Vajngerl <qui...@gmail.com>
CommitDate: Thu Dec 28 02:36:29 2023 +0100

    editeng: move ESelection in its own header file
    
    Change-Id: I46fbaf9fea8308169f2ada19787684c87b0c3c5b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161338
    Tested-by: Jenkins
    Reviewed-by: Tomaž Vajngerl <qui...@gmail.com>

diff --git a/include/editeng/ESelection.hxx b/include/editeng/ESelection.hxx
new file mode 100644
index 000000000000..2ddc90c997d8
--- /dev/null
+++ b/include/editeng/ESelection.hxx
@@ -0,0 +1,120 @@
+/* -*- 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 .
+ */
+
+#pragma once
+
+struct ESelection
+{
+    sal_Int32 nStartPara;
+    sal_Int32 nStartPos;
+    sal_Int32 nEndPara;
+    sal_Int32 nEndPos;
+
+    ESelection()
+        : nStartPara(0)
+        , nStartPos(0)
+        , nEndPara(0)
+        , nEndPos(0)
+    {
+    }
+
+    ESelection(sal_Int32 nStPara, sal_Int32 nStPos, sal_Int32 nEPara, 
sal_Int32 nEPos)
+        : nStartPara(nStPara)
+        , nStartPos(nStPos)
+        , nEndPara(nEPara)
+        , nEndPos(nEPos)
+    {
+    }
+
+    ESelection(sal_Int32 nPara, sal_Int32 nPos)
+        : nStartPara(nPara)
+        , nStartPos(nPos)
+        , nEndPara(nPara)
+        , nEndPos(nPos)
+    {
+    }
+
+    void Adjust();
+    bool operator==(const ESelection& rS) const;
+    bool operator!=(const ESelection& rS) const { return !operator==(rS); }
+    bool operator<(const ESelection& rS) const;
+    bool operator>(const ESelection& rS) const;
+    bool IsZero() const;
+    bool HasRange() const;
+};
+
+template <typename charT, typename traits>
+inline std::basic_ostream<charT, traits>& operator<<(std::basic_ostream<charT, 
traits>& stream,
+                                                     ESelection const& sel)
+{
+    return stream << "ESelection(" << sel.nStartPara << ',' << sel.nStartPos 
<< "," << sel.nEndPara
+                  << "," << sel.nEndPos << ")";
+}
+
+inline bool ESelection::HasRange() const
+{
+    return (nStartPara != nEndPara) || (nStartPos != nEndPos);
+}
+
+inline bool ESelection::IsZero() const
+{
+    return ((nStartPara == 0) && (nStartPos == 0) && (nEndPara == 0) && 
(nEndPos == 0));
+}
+
+inline bool ESelection::operator==(const ESelection& rS) const
+{
+    return ((nStartPara == rS.nStartPara) && (nStartPos == rS.nStartPos)
+            && (nEndPara == rS.nEndPara) && (nEndPos == rS.nEndPos));
+}
+
+inline bool ESelection::operator<(const ESelection& rS) const
+{
+    // The selection must be adjusted.
+    // => Only check if end of 'this' < Start of rS
+    return (nEndPara < rS.nStartPara)
+           || ((nEndPara == rS.nStartPara) && (nEndPos < rS.nStartPos) && 
!operator==(rS));
+}
+
+inline bool ESelection::operator>(const ESelection& rS) const
+{
+    // The selection must be adjusted.
+    // => Only check if end of 'this' < Start of rS
+    return (nStartPara > rS.nEndPara)
+           || ((nStartPara == rS.nEndPara) && (nStartPos > rS.nEndPos) && 
!operator==(rS));
+}
+
+inline void ESelection::Adjust()
+{
+    bool bSwap = false;
+    if (nStartPara > nEndPara)
+        bSwap = true;
+    else if ((nStartPara == nEndPara) && (nStartPos > nEndPos))
+        bSwap = true;
+
+    if (bSwap)
+    {
+        sal_Int32 nSPar = nStartPara;
+        sal_Int32 nSPos = nStartPos;
+        nStartPara = nEndPara;
+        nStartPos = nEndPos;
+        nEndPara = nSPar;
+        nEndPos = nSPos;
+    }
+}
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/include/editeng/editdata.hxx b/include/editeng/editdata.hxx
index 1762fd86e837..c31fc7eb7bd1 100644
--- a/include/editeng/editdata.hxx
+++ b/include/editeng/editdata.hxx
@@ -24,6 +24,7 @@
 #include <rtl/ustring.hxx>
 #include <editeng/editengdllapi.h>
 #include <i18nlangtag/lang.h>
+#include <editeng/ESelection.hxx>
 #include <memory>
 #include <ostream>
 
@@ -108,95 +109,6 @@ inline std::basic_ostream<charT, traits> & operator <<(
     return stream << "EPosition(" << pos.nPara << ',' << pos.nIndex << ")";
 }
 
-struct ESelection
-{
-    sal_Int32   nStartPara;
-    sal_Int32   nStartPos;
-    sal_Int32   nEndPara;
-    sal_Int32   nEndPos;
-
-    ESelection() : nStartPara( 0 ), nStartPos( 0 ), nEndPara( 0 ), nEndPos( 0 
) {}
-
-    ESelection( sal_Int32 nStPara, sal_Int32 nStPos,
-                sal_Int32 nEPara, sal_Int32 nEPos )
-        : nStartPara( nStPara )
-        , nStartPos( nStPos )
-        , nEndPara( nEPara )
-        , nEndPos( nEPos )
-        { }
-
-    ESelection( sal_Int32 nPara, sal_Int32 nPos )
-        : nStartPara( nPara )
-        , nStartPos( nPos )
-        , nEndPara( nPara )
-        , nEndPos( nPos )
-        { }
-
-    void    Adjust();
-    bool    operator==( const ESelection& rS ) const;
-    bool    operator!=( const ESelection& rS ) const { return !operator==(rS); 
}
-    bool    operator<( const ESelection& rS ) const;
-    bool    operator>( const ESelection& rS ) const;
-    bool    IsZero() const;
-    bool    HasRange() const;
-};
-
-template<typename charT, typename traits>
-inline std::basic_ostream<charT, traits> & operator <<(
-    std::basic_ostream<charT, traits> & stream, ESelection const& sel)
-{
-    return stream << "ESelection(" << sel.nStartPara << ',' << sel.nStartPos 
<< "," << sel.nEndPara << "," << sel.nEndPos << ")";
-}
-
-inline bool ESelection::HasRange() const
-{
-    return ( nStartPara != nEndPara ) || ( nStartPos != nEndPos );
-}
-
-inline bool ESelection::IsZero() const
-{
-    return ( ( nStartPara == 0 ) && ( nStartPos == 0 ) &&
-             ( nEndPara == 0 ) && ( nEndPos == 0 ) );
-}
-
-inline bool ESelection::operator==( const ESelection& rS ) const
-{
-    return ( ( nStartPara == rS.nStartPara ) && ( nStartPos == rS.nStartPos ) 
&&
-             ( nEndPara == rS.nEndPara ) && ( nEndPos == rS.nEndPos ) );
-}
-
-inline bool ESelection::operator<( const ESelection& rS ) const
-{
-    // The selection must be adjusted.
-    // => Only check if end of 'this' < Start of rS
-    return ( nEndPara < rS.nStartPara ) ||
-        ( ( nEndPara == rS.nStartPara ) && ( nEndPos < rS.nStartPos ) && 
!operator==( rS ) );
-}
-
-inline bool ESelection::operator>( const ESelection& rS ) const
-{
-    // The selection must be adjusted.
-    // => Only check if end of 'this' < Start of rS
-    return ( nStartPara > rS.nEndPara ) ||
-        ( ( nStartPara == rS.nEndPara ) && ( nStartPos > rS.nEndPos ) && 
!operator==( rS ) );
-}
-
-inline void ESelection::Adjust()
-{
-    bool bSwap = false;
-    if ( nStartPara > nEndPara )
-        bSwap = true;
-    else if ( ( nStartPara == nEndPara ) && ( nStartPos > nEndPos ) )
-        bSwap = true;
-
-    if ( bSwap )
-    {
-        sal_Int32  nSPar = nStartPara; sal_Int32 nSPos = nStartPos;
-        nStartPara = nEndPara; nStartPos = nEndPos;
-        nEndPara = nSPar; nEndPos = nSPos;
-    }
-}
-
 struct EDITENG_DLLPUBLIC EFieldInfo
 {
     std::unique_ptr<SvxFieldItem>   pFieldItem;

Reply via email to