forms/source/component/FormComponent.cxx         |    6 +-
 forms/source/component/FormattedField.cxx        |   67 +++++++++++------------
 forms/source/component/FormattedField.hxx        |    2 
 forms/source/component/FormattedFieldWrapper.hxx |    2 
 forms/source/component/FormsCollection.hxx       |   12 +---
 forms/source/component/Grid.hxx                  |   16 ++---
 forms/source/component/GroupBox.cxx              |    1 
 forms/source/component/GroupBox.hxx              |    2 
 forms/source/component/GroupManager.cxx          |   35 +++++-------
 forms/source/component/GroupManager.hxx          |   34 +++++------
 forms/source/component/Hidden.cxx                |    2 
 forms/source/component/ImageButton.cxx           |   10 +--
 forms/source/component/ImageButton.hxx           |    4 -
 13 files changed, 92 insertions(+), 101 deletions(-)

New commits:
commit 0f8ed329131573289c98b8339f53e57abf67967d
Author: Philipp Weissenbacher <p.weissenbac...@gmail.com>
Date:   Wed Sep 24 11:43:17 2014 +0200

    fdo#39468 Translate German comments, clean some ws
    
    Conflicts:
        forms/source/component/FormComponent.cxx
        forms/source/component/FormattedField.cxx
    
    Change-Id: I76dacfff3aa6f5680a1e81c97281c4f744ec1ceb
    Reviewed-on: https://gerrit.libreoffice.org/11622
    Reviewed-by: Caolán McNamara <caol...@redhat.com>
    Tested-by: Caolán McNamara <caol...@redhat.com>

diff --git a/forms/source/component/FormComponent.cxx 
b/forms/source/component/FormComponent.cxx
index e8547ac..996e1b6 100644
--- a/forms/source/component/FormComponent.cxx
+++ b/forms/source/component/FormComponent.cxx
@@ -154,7 +154,7 @@ void OControl::doSetDelegator()
     decrement( m_refCount );
 }
 
-// UNO Anbindung
+// UNO Binding
 Any SAL_CALL OControl::queryAggregation( const Type& _rType ) 
throw(RuntimeException, std::exception)
 {
     // ask the base class
@@ -758,13 +758,13 @@ void SAL_CALL OControlModel::write(const 
Reference<css::io::XObjectOutputStream>
     xMark->jumpToFurthest();
     xMark->deleteMark(nMark);
 
-    // 2. wrting a version number
+    // 2. writing a version number
     _rxOutStream->writeShort(0x0003);
 
     // 3. writing the general properties
     ::comphelper::operator<<( _rxOutStream, m_aName);
     _rxOutStream->writeShort(m_nTabIndex);
-    ::comphelper::operator<<( _rxOutStream, m_aTag); // 3. version
+    ::comphelper::operator<<( _rxOutStream, m_aTag); // 3rd version
 
     // IMPORTANT NOTE!
     // don't write any new members here: this wouldn't be compatible with 
older versions, as OControlModel
diff --git a/forms/source/component/FormattedField.cxx 
b/forms/source/component/FormattedField.cxx
index 30d72fc..6ed7274 100644
--- a/forms/source/component/FormattedField.cxx
+++ b/forms/source/component/FormattedField.cxx
@@ -196,7 +196,7 @@ void OFormattedControl::keyPressed(const 
::com::sun::star::awt::KeyEvent& e) thr
 {
     if( e.KeyCode != KEY_RETURN || e.Modifiers != 0 )
         return;
-    // Is the control on a form with a submit URL?
+    // Is the control located in a form with a Submit URL?
     Reference<com::sun::star::beans::XPropertySet>  xSet(getModel(), 
UNO_QUERY);
     if( !xSet.is() )
         return;
@@ -223,13 +223,13 @@ void OFormattedControl::keyPressed(const 
::com::sun::star::awt::KeyEvent& e) thr
             if (hasProperty(PROPERTY_CLASSID, xFCSet) &&
                 getINT16(xFCSet->getPropertyValue(PROPERTY_CLASSID)) == 
FormComponentType::TEXTFIELD)
             {
-                // Found another edit ==> no submit
+                // Found another Edit -> Do not submit then
                 if (xFCSet != xSet)
                     return;
             }
         }
     }
-    // Still in the handler, trigger submit asynchronous
+    // Because we're still in the Handler, execute submit asynchronously
     if( m_nKeyEvent )
         Application::RemoveUserEvent( m_nKeyEvent );
     m_nKeyEvent = Application::PostUserEvent( LINK(this, OFormattedControl,
@@ -283,7 +283,7 @@ void OFormattedModel::implConstruct()
 }
 OFormattedModel::OFormattedModel(const Reference<XComponentContext>& 
_rxFactory)
     :OEditBaseModel(_rxFactory, VCL_CONTROLMODEL_FORMATTEDFIELD, 
FRM_SUN_CONTROL_FORMATTEDFIELD, true, true )
-                            // use the old control name for compytibility 
reasons
+    // use the old control name for compytibility reasons
     ,OErrorBroadcaster( OComponentHelper::rBHelper )
 {
     implConstruct();
@@ -363,17 +363,17 @@ void OFormattedModel::describeFixedProperties( Sequence< 
Property >& _rProps ) c
 void OFormattedModel::describeAggregateProperties( Sequence< Property >& 
_rAggregateProps ) const
 {
     OEditBaseModel::describeAggregateProperties( _rAggregateProps );
-    // TreatAsNumeric not transient : we want to bind it to the UI (necessary, 
because EffectiveDefault
-    // - could be text or numbers - for making sense)
+    // TreatAsNumeric is not transient: we want to attach it to the UI
+    // This is necessary to make EffectiveDefault (which may be text or a 
number) meaningful
     ModifyPropertyAttributes(_rAggregateProps, PROPERTY_TREATASNUMERIC, 0, 
PropertyAttribute::TRANSIENT);
-    // same for FormatKey
+    // Same for FormatKey
     // (though the paragraph above for the TreatAsNumeric does not hold 
anymore - we do not have an UI for this.
     // But we have for the format key ...)
     ModifyPropertyAttributes(_rAggregateProps, PROPERTY_FORMATKEY, 0, 
PropertyAttribute::TRANSIENT);
     RemoveProperty(_rAggregateProps, PROPERTY_STRICTFORMAT);
-        // no strict format property for formatted fields: it does not make 
sense, 'cause
-        // there is no general way to decide which characters/sub strings are 
allowed during the input of an
-        // arbitraryly formatted control
+    // no strict format property for formatted fields: it does not make sense, 
'cause
+    // there is no general way to decide which characters/sub strings are 
allowed during the input of an
+    // arbitraryly formatted control
 }
 
 void OFormattedModel::getFastPropertyValue(Any& rValue, sal_Int32 nHandle) 
const
@@ -492,7 +492,7 @@ Reference< XNumberFormatsSupplier > 
OFormattedModel::calcFormatsSupplier() const
 {
     Reference<XNumberFormatsSupplier>  xSupplier;
     DBG_ASSERT(m_xAggregateSet.is(), "OFormattedModel::calcFormatsSupplier : 
have no aggregate !");
-    // check if my aggregated model has a FormatSupplier
+    // Does my aggregate model have a FormatSupplier?
     if( m_xAggregateSet.is() )
         m_xAggregateSet->getPropertyValue(PROPERTY_FORMATSSUPPLIER) >>= 
xSupplier;
     if (!xSupplier.is())
@@ -501,7 +501,7 @@ Reference< XNumberFormatsSupplier > 
OFormattedModel::calcFormatsSupplier() const
     if (!xSupplier.is())
         xSupplier = calcDefaultFormatsSupplier();
     DBG_ASSERT(xSupplier.is(), "OFormattedModel::calcFormatsSupplier : no 
supplier !");
-        // now it should be there
+    // We should have one by now
     return xSupplier;
 }
 
@@ -509,9 +509,9 @@ Reference<XNumberFormatsSupplier>  
OFormattedModel::calcFormFormatsSupplier() co
 {
     Reference<XChild>  xMe;
     query_interface(static_cast<XWeak*>(const_cast<OFormattedModel*>(this)), 
xMe);
-    // now we are sure, in case of an aggregation, to get the right object
+    // By this we make sure that we get the right object even when aggregating
     DBG_ASSERT(xMe.is(), "OFormattedModel::calcFormFormatsSupplier : I should 
have a content interface !");
-    // expand to the top, till we get to an initial form (starting with my own 
parent)
+    // Iterate through until we reach a StartForm (starting with an own Parent)
     Reference<XChild>  xParent(xMe->getParent(), UNO_QUERY);
     Reference<XForm>  xNextParentForm(xParent, UNO_QUERY);
     while (!xNextParentForm.is() && xParent.is())
@@ -524,7 +524,7 @@ Reference<XNumberFormatsSupplier>  
OFormattedModel::calcFormFormatsSupplier() co
         OSL_FAIL("OFormattedModel::calcFormFormatsSupplier : have no ancestor 
which is a form !");
         return NULL;
     }
-    // the FormatSupplier of my ancestor (if he owns one)
+    // The FormatSupplier of my ancestor (if it has one)
     Reference< XRowSet > xRowSet( xNextParentForm, UNO_QUERY );
     Reference< XNumberFormatsSupplier > xSupplier;
     if (xRowSet.is())
@@ -540,7 +540,7 @@ Reference< XNumberFormatsSupplier > 
OFormattedModel::calcDefaultFormatsSupplier(
 // XBoundComponent
 void OFormattedModel::loaded(const EventObject& rEvent) throw ( 
::com::sun::star::uno::RuntimeException, std::exception)
 {
-    // HACK : our onConnectedDbColumn accesses our NumberFormatter which locks 
the solar mutex (as it doesn't have
+    // HACK: our onConnectedDbColumn accesses our NumberFormatter which locks 
the solar mutex (as it doesn't have
     // an own one). To prevent deadlocks with other threads which may request 
a property from us in an
     // UI-triggered action (e.g. an tooltip) we lock the solar mutex _here_ 
before our base class locks
     // it's own muext (which is used for property requests)
@@ -566,7 +566,7 @@ void OFormattedModel::onConnectedDbColumn( const Reference< 
XInterface >& _rxFor
     {   // all the following doesn't make any sense if we have no aggregate ...
         Any aSupplier = 
m_xAggregateSet->getPropertyValue(PROPERTY_FORMATSSUPPLIER);
         DBG_ASSERT( aSupplier.hasValue(), 
"OFormattedModel::onConnectedDbColumn : invalid property value !" );
-        // this should already be initiated and correctly set by the 
constructor or the read function
+        // This should've been set to the correct value in the ctor or in the 
read
         Any aFmtKey = m_xAggregateSet->getPropertyValue(PROPERTY_FORMATKEY);
         if ( !(aFmtKey >>= nFormatKey ) )
         {   // nobody gave us a format to use. So we examine the field we're 
bound to for a
@@ -598,7 +598,7 @@ void OFormattedModel::onConnectedDbColumn( const Reference< 
XInterface >& _rxFor
                 aSupplier >>= m_xOriginalFormatter;
                 m_xAggregateSet->setPropertyValue(PROPERTY_FORMATSSUPPLIER, 
makeAny(xSupplier));
                 m_xAggregateSet->setPropertyValue(PROPERTY_FORMATKEY, aFmtKey);
-                // adapt the numeric-flag to my bound field
+                // Adapt the NumericFalg to my bound field
                 if (xField.is())
                 {
                     m_bNumeric = false;
@@ -640,7 +640,7 @@ void OFormattedModel::onDisconnectedDbColumn()
 {
     OEditBaseModel::onDisconnectedDbColumn();
     if (m_xOriginalFormatter.is())
-    {   // the aggregated model has no format information
+    {   // Our aggregated model does not hold any Format information
         m_xAggregateSet->setPropertyValue(PROPERTY_FORMATSSUPPLIER, 
makeAny(m_xOriginalFormatter));
         m_xAggregateSet->setPropertyValue(PROPERTY_FORMATKEY, Any());
         setPropertyValue(PROPERTY_TREATASNUMERIC, makeAny(m_bOriginalNumeric));
@@ -656,8 +656,9 @@ void OFormattedModel::write(const 
Reference<XObjectOutputStream>& _rxOutStream)
     OEditBaseModel::write(_rxOutStream);
     _rxOutStream->writeShort(0x0003);
     DBG_ASSERT(m_xAggregateSet.is(), "OFormattedModel::write : have no 
aggregate !");
-    // take my format (possibly void) into a persistent format (the supplier 
together with the key
-    // would also be persistent, but saving the whole supplier would be 
overhead)
+    // Bring my Format (may be void) to a persistent Format.
+    // The Supplier together with the Key is already persistent, but that 
doesn't mean
+    // we have to save the Supplier (which would be quite some overhead)
         Reference<XNumberFormatsSupplier>  xSupplier;
         Any aFmtKey;
     bool bVoidKey = true;
@@ -670,12 +671,12 @@ void OFormattedModel::write(const 
Reference<XObjectOutputStream>& _rxOutStream)
         }
         aFmtKey = m_xAggregateSet->getPropertyValue(PROPERTY_FORMATKEY);
         bVoidKey = (!xSupplier.is() || !aFmtKey.hasValue()) || (isLoaded() && 
m_xOriginalFormatter.is());
-            // (no formatter and/or key) or (loaded and fakes formatter)
+        // (no Format and/or Key) OR (loaded and faked Formatter)
     }
     _rxOutStream->writeBoolean(!bVoidKey);
     if (!bVoidKey)
     {
-        // build persistent data with the FormatKey and the formatter
+        // Create persistent values from the FormatKey and the Formatter
         Any aKey = m_xAggregateSet->getPropertyValue(PROPERTY_FORMATKEY);
         sal_Int32 nKey = aKey.hasValue() ? getINT32(aKey) : 0;
         Reference<XNumberFormats>  xFormats = xSupplier->getNumberFormats();
@@ -702,8 +703,8 @@ void OFormattedModel::write(const 
Reference<XObjectOutputStream>& _rxOutStream)
     // version 2 : write the properties common to all OEditBaseModels
     writeCommonEditProperties(_rxOutStream);
     // version 3 : write the effective value property of the aggregate
-    // Due to a bug within the UnoControlFormattedFieldModel implementation 
(our default aggregate) this props value isn't correctly read
-    // and this can't be corrected without being incompatible.
+    // Due to a bug within the UnoControlFormattedFieldModel implementation 
(our default aggregate)
+    // this props value isn't correctly read and this can't be corrected 
without being incompatible.
     // so we have our own handling.
     // and to be a little bit more compatible we make the following section 
skippable
     {
@@ -711,7 +712,7 @@ void OFormattedModel::write(const 
Reference<XObjectOutputStream>& _rxOutStream)
         // a sub version within the skippable block
         _rxOutStream->writeShort(0x0000);
         // version 0: the effective value of the aggregate
-                Any aEffectiveValue;
+        Any aEffectiveValue;
         if (m_xAggregateSet.is())
         {
             try { aEffectiveValue = 
m_xAggregateSet->getPropertyValue(PROPERTY_EFFECTIVE_VALUE); } catch(const 
Exception&) { }
@@ -757,7 +758,7 @@ void OFormattedModel::read(const 
Reference<XObjectInputStream>& _rxInStream) thr
                 LanguageType eDescriptionLanguage = 
(LanguageType)_rxInStream->readLong();
                 // and let a formatter roll dice based on that to create a 
key...
                 xSupplier = calcFormatsSupplier();
-                    // calcFormatsSupplier first takes the one from the model, 
then one from the starform, then a new one...
+                // calcFormatsSupplier first takes the one from the model, 
then one from the starform, then a new one...
                 Reference<XNumberFormats>  xFormats = 
xSupplier->getNumberFormats();
                 if (xFormats.is())
                 {
@@ -953,9 +954,9 @@ Any OFormattedModel::translateControlValueToExternalValue( 
) const
     {
         double fValue = 0;
         OSL_VERIFY( aControlValue >>= fValue );
-            // if this asserts ... well, the somebody set the TreatAsNumeric 
property to false,
-            // and the control value is a string. This implies some weird 
misconfiguration
-            // of the FormattedModel, so we won't care for it for the moment.
+        // if this asserts ... well, the somebody set the TreatAsNumeric 
property to false,
+        // and the control value is a string. This implies some weird 
misconfiguration
+        // of the FormattedModel, so we won't care for it for the moment.
         aExternalValue <<= fValue != 0.0;
     }
     break;
@@ -963,9 +964,9 @@ Any OFormattedModel::translateControlValueToExternalValue( 
) const
     {
         double fValue = 0;
         OSL_VERIFY( aControlValue >>= fValue );
-            // if this asserts ... well, the somebody set the TreatAsNumeric 
property to false,
-            // and the control value is a string. This implies some weird 
misconfiguration
-            // of the FormattedModel, so we won't care for it for the moment.
+        // if this asserts ... well, the somebody set the TreatAsNumeric 
property to false,
+        // and the control value is a string. This implies some weird 
misconfiguration
+        // of the FormattedModel, so we won't care for it for the moment.
         if ( aExternalValueType.equals( cppu::UnoType< UNODate >::get() ) )
         {
             aExternalValue <<= DBTypeConversion::toDate( fValue, m_aNullDate );
diff --git a/forms/source/component/FormattedField.hxx 
b/forms/source/component/FormattedField.hxx
index 7c9c261..c4b047b 100644
--- a/forms/source/component/FormattedField.hxx
+++ b/forms/source/component/FormattedField.hxx
@@ -44,7 +44,7 @@ class OFormattedModel
 
         sal_Int32                           m_nFieldType;
         sal_Int16                           m_nKeyType;
-        bool                            m_bOriginalNumeric      : 1,
+        bool                                m_bOriginalNumeric      : 1,
                                             m_bNumeric              : 1;    // 
analogous for the TreatAsNumeric-property
 
     protected:
diff --git a/forms/source/component/FormattedFieldWrapper.hxx 
b/forms/source/component/FormattedFieldWrapper.hxx
index 1315d4f..1ea2290 100644
--- a/forms/source/component/FormattedFieldWrapper.hxx
+++ b/forms/source/component/FormattedFieldWrapper.hxx
@@ -46,7 +46,7 @@ protected:
     ::com::sun::star::uno::Reference< ::com::sun::star::uno::XAggregation>     
 m_xAggregate;
 
     rtl::Reference< OEditModel > m_pEditPart;
-        // if we act as formatted this is used to write the EditModel part
+    // if we act as formatted this is used to write the EditModel part
     ::com::sun::star::uno::Reference< ::com::sun::star::io::XPersistObject>    
 m_xFormattedPart;
     // if we act as formatted, this is the PersistObject interface of our 
aggregate, used
     // to read and write the FormattedModel part
diff --git a/forms/source/component/FormsCollection.hxx 
b/forms/source/component/FormsCollection.hxx
index 3980417..7344baa 100644
--- a/forms/source/component/FormsCollection.hxx
+++ b/forms/source/component/FormsCollection.hxx
@@ -31,26 +31,22 @@
 namespace frm
 {
 
-
-
 // OFormsCollection
-// Implementiert den UNO-Container fuer Formulare
-// enthaelt alle zugeordneten Forms
-// dieses Container kann selbst den Context fuer Formulare darstellen
-// oder aussen einen Context uebergeben bekommen
+// Implements the UNO Container for Forms and contains all assigend Forms.
+// It can either represent the Context for Forms or be passed a Context.
 
 typedef ::cppu::OComponentHelper FormsCollectionComponentBase;
 typedef ::cppu::ImplHelper2<    ::com::sun::star::form::XForms
                                 ,::com::sun::star::lang::XServiceInfo > 
OFormsCollection_BASE;
 
-    // else MSVC kills itself on some statements
+// else MSVC kills itself on some statements
 class OFormsCollection
         :public FormsCollectionComponentBase
         ,public OInterfaceContainer
         ,public OFormsCollection_BASE
 {
     ::osl::Mutex                m_aMutex;
-    ::comphelper::InterfaceRef  m_xParent;          // Parent
+    ::comphelper::InterfaceRef  m_xParent; // Parent
 
 public:
     OFormsCollection(const ::com::sun::star::uno::Reference< 
::com::sun::star::uno::XComponentContext>& _rxFactory);
diff --git a/forms/source/component/Grid.hxx b/forms/source/component/Grid.hxx
index 3c03911..d48f8e1 100644
--- a/forms/source/component/Grid.hxx
+++ b/forms/source/component/Grid.hxx
@@ -43,7 +43,7 @@ class OGridColumn;
     struct ColumnDescription : public ElementDescription
     {
     public:
-        OGridColumn*                pColumn;        // not owned by this 
instance! only to prevent duplicate XUnoTunnel usage
+        OGridColumn*                pColumn; // not owned by this instance! 
only to prevent duplicate XUnoTunnel usage
     };
 
 
@@ -69,10 +69,10 @@ class OGridControlModel :public OControlModel
                                             m_aRowSetChangeListeners;
 
 // [properties]
-    ::com::sun::star::uno::Any              m_aRowHeight;           // 
Zeilenhoehe
+    ::com::sun::star::uno::Any              m_aRowHeight; // Row height
     ::com::sun::star::uno::Any              m_aTabStop;
     ::com::sun::star::uno::Any              m_aBackgroundColor;
-    ::com::sun::star::uno::Any              m_aCursorColor;             // 
transient
+    ::com::sun::star::uno::Any              m_aCursorColor; // transient
     ::com::sun::star::uno::Any              m_aBorderColor;
     OUString                         m_aDefaultControl;
     OUString                         m_sHelpText;
@@ -81,10 +81,10 @@ class OGridControlModel :public OControlModel
     ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet >  
     m_xSelection;
 
 // [properties]
-    OUString             m_sHelpURL;                 // URL
-    sal_Int16                   m_nBorder;
-    sal_Int16                   m_nWritingMode;
-    sal_Int16                   m_nContextWritingMode;
+    OUString                m_sHelpURL;                 // URL
+    sal_Int16               m_nBorder;
+    sal_Int16               m_nWritingMode;
+    sal_Int16               m_nContextWritingMode;
     bool                    m_bEnableVisible    : 1;
     bool                    m_bEnable           : 1;
     bool                    m_bNavigation       : 1;
@@ -100,7 +100,7 @@ protected:
 public:
     DECLARE_DEFAULT_LEAF_XTOR( OGridControlModel );
 
-    // UNO Anbindung
+    // UNO Binding
     DECLARE_UNO3_AGG_DEFAULTS(OGridControlModel, OControlModel)
     virtual ::com::sun::star::uno::Any SAL_CALL queryAggregation( const 
::com::sun::star::uno::Type& _rType ) throw 
(::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
 
diff --git a/forms/source/component/GroupBox.cxx 
b/forms/source/component/GroupBox.cxx
index 9e9b596..b0e5109 100644
--- a/forms/source/component/GroupBox.cxx
+++ b/forms/source/component/GroupBox.cxx
@@ -116,7 +116,6 @@ void SAL_CALL OGroupBoxModel::read(const Reference< 
XObjectInputStream>& _rxInSt
     // Version
     sal_uInt16 nVersion = _rxInStream->readShort();
     DBG_ASSERT(nVersion > 0, "OGroupBoxModel::read : version 0 ? this should 
never have been written !");
-        // ups, ist das Englisch richtig ? ;)
 
     if (nVersion == 2)
         readHelpTextCompatibly(_rxInStream);
diff --git a/forms/source/component/GroupBox.hxx 
b/forms/source/component/GroupBox.hxx
index 1fd1777..255f705 100644
--- a/forms/source/component/GroupBox.hxx
+++ b/forms/source/component/GroupBox.hxx
@@ -53,7 +53,7 @@ protected:
 };
 
 
-// OGroupBoxControl (nur aus Kompatibilitaet zur 5.0)
+// OGroupBoxControl (only for compatibility for 5.0)
 
 class OGroupBoxControl : public OControl
 {
diff --git a/forms/source/component/GroupManager.cxx 
b/forms/source/component/GroupManager.cxx
index 2a5a984..d25a6db 100644
--- a/forms/source/component/GroupManager.cxx
+++ b/forms/source/component/GroupManager.cxx
@@ -117,7 +117,7 @@ public:
     bool operator() (const OGroupComp& lhs, const OGroupComp& rhs) const
     {
         bool bResult;
-        // TabIndex von 0 wird hinten einsortiert
+        // TabIndex of 0 will be added at the end
         if (lhs.m_nTabIndex == rhs.GetTabIndex())
             bResult = lhs.m_nPos < rhs.GetPos();
         else if (lhs.m_nTabIndex && rhs.GetTabIndex())
@@ -164,20 +164,19 @@ void OGroup::RemoveComponent( const 
Reference<XPropertySet>& rxElement )
             m_aCompArray.erase( m_aCompArray.begin() + nGroupCompPos );
 
             /*
-            Durch das Entfernen der GroupComp ist die Einfuegeposition
-            ungueltig geworden. Sie braucht hier aber nicht angepasst werden,
-            da sie fortlaufend vergeben wird und damit immer
-            aufsteigend eindeutig ist.
-            */
+             * By removing the GroupComp the insertion position has become 
invalid.
+             * We do not to change it here, however, because its passed on 
continously
+             * and ascending distinctively.
+             */
         }
         else
         {
-            OSL_FAIL( "OGroup::RemoveComponent: Component nicht in Gruppe" );
+            OSL_FAIL( "OGroup::RemoveComponent: Component not in Group" );
         }
     }
     else
     {
-        OSL_FAIL( "OGroup::RemoveComponent: Component nicht in Gruppe" );
+        OSL_FAIL( "OGroup::RemoveComponent: Component not in Group" );
     }
 }
 
@@ -243,7 +242,7 @@ void OGroupManager::removeFromGroupMap(const OUString& 
_sGroupName,const Referen
         // group exists
         aFind->second.RemoveComponent( _xSet );
 
-        // Wenn Anzahl der Gruppenelemente == 1 ist, Gruppe deaktivieren
+        // If the count of Group elements == 1 -> deactivate Group
         sal_Int32 nCount = aFind->second.Count();
         if ( nCount == 1 || nCount == 0 )
         {
@@ -263,7 +262,7 @@ void OGroupManager::removeFromGroupMap(const OUString& 
_sGroupName,const Referen
     }
 
 
-    // Bei Component als PropertyChangeListener abmelden
+    // Deregister as PropertyChangeListener at Component
     _xSet->removePropertyChangeListener( PROPERTY_NAME, this );
     if (hasProperty(PROPERTY_GROUP_NAME, _xSet))
         _xSet->removePropertyChangeListener( PROPERTY_GROUP_NAME, this );
@@ -297,7 +296,7 @@ void SAL_CALL OGroupManager::propertyChange(const 
PropertyChangeEvent& evt) thro
 
     removeFromGroupMap(sGroupName,xSet);
 
-    // Component neu einordnen
+    // Re-insert Component
     InsertElement( xSet );
 }
 
@@ -359,15 +358,15 @@ void OGroupManager::getGroupByName(const OUString& 
_rName, Sequence< Reference<X
 
 void OGroupManager::InsertElement( const Reference<XPropertySet>& xSet )
 {
-    // Nur ControlModels
+    // Only ControlModels
     Reference<XControlModel>  xControl(xSet, UNO_QUERY);
     if (!xControl.is() )
         return;
 
-    // Component in CompGroup aufnehmen
+    // Add Component to CompGroup
     m_pCompGroup->InsertComponent( xSet );
 
-    // Component in Gruppe aufnehmen
+    // Add Component to Group
     OUString sGroupName( GetGroupName( xSet ) );
 
     OGroupArr::iterator aFind = m_aGroupArr.find(sGroupName);
@@ -403,24 +402,24 @@ void OGroupManager::InsertElement( const 
Reference<XPropertySet>& xSet )
             m_aActiveGroupMap.push_back(  aFind );
     }
 
-    // Bei Component als PropertyChangeListener anmelden
+    // Register as PropertyChangeListener at Component
     xSet->addPropertyChangeListener( PROPERTY_NAME, this );
     if (hasProperty(PROPERTY_GROUP_NAME, xSet))
         xSet->addPropertyChangeListener( PROPERTY_GROUP_NAME, this );
 
-    // Tabindex muss nicht jeder unterstuetzen
+    // Not everyone needs to support Tabindex
     if (hasProperty(PROPERTY_TABINDEX, xSet))
         xSet->addPropertyChangeListener( PROPERTY_TABINDEX, this );
 }
 
 void OGroupManager::RemoveElement( const Reference<XPropertySet>& xSet )
 {
-    // Nur ControlModels
+    // Only ControlModels
     Reference<XControlModel>  xControl(xSet, UNO_QUERY);
     if (!xControl.is() )
         return;
 
-    // Component aus Gruppe entfernen
+    // Remove Component from Group
     OUString     sGroupName( GetGroupName( xSet ) );
 
     removeFromGroupMap(sGroupName,xSet);
diff --git a/forms/source/component/GroupManager.hxx 
b/forms/source/component/GroupManager.hxx
index ca686d4..01d8c60 100644
--- a/forms/source/component/GroupManager.hxx
+++ b/forms/source/component/GroupManager.hxx
@@ -36,22 +36,19 @@
 using namespace comphelper;
 
 /*
- * Der GroupManager horcht an der starform, ob FormComponents eingefuegt oder 
entfernt
- * werden. Zusaetzlich horcht er bei den FormComponents an den Properties
- * "Name" und "TabIndex". Mit diesen Infos aktualisiert er seine Gruppen.
+ * The GroupManager listens at the StarForm for FormComponent insertion and 
removal as well as
+ * its properties "Name" and "TabIndex" and updates its Group using this 
information.
  *
- * Der GroupManager verwaltet eine Gruppe, in der alle Controls nach TabIndex
- * geordnet sind, und ein Array von Gruppen, in dem jede FormComponent noch
- * einmal einer Gruppe dem Namen nach zugeordnet wird.
- * Die einzelnen Gruppen werden ueber eine Map aktiviert, wenn sie mehr als
- * ein Element besitzen.
+ * The GroupManager manages a Group in which all Controls are sorted by 
TabIndex.
+ * It also manages an array of Groups, in which each FormComponent is assigned 
a
+ * Group according to its name.
+ * Each Group is activated using a Map, if they contain more than one element.
  *
- * Die Gruppen verwalten intern die FormComponents in zwei Arrays. In dem
- * GroupCompArray werden die Components nach TabIndex und Einfuegepostion
- * sortiert. Da auf dieses Array ueber die FormComponent zugegriffen
- * wird, gibt es noch das GroupCompAccessArray, in dem die FormComponents
- * nach ihrer Speicheradresse sortiert sind. Jedes Element des
- * GroupCompAccessArrays ist mit einem Element des GroupCompArrays verpointert.
+ * The Groups manage the FormComponents internally using two arrays.
+ * In the GroupCompArray the Components are sorted by TabIndex and insertion 
position.
+ * Because this array is accessed via the FormComponent, we also have the 
GroupCompAccessArray
+ * in which the FormComponents are sorted by their storage address.
+ * Every element of the GroupCompArray has a pointer to the GroupCompArray.
  */
 namespace frm
 {
@@ -144,7 +141,7 @@ class OGroup
     OGroupCompAccArr    m_aCompAccArray;
 
     OUString m_aGroupName;
-    sal_uInt16  m_nInsertPos;               // Die Einfugeposition der 
GroupComps wird von der Gruppe bestimmt.
+    sal_uInt16  m_nInsertPos; // The insertion position of the GroupComps is 
determind by the Group
 
     friend class OGroupLess;
 
@@ -170,10 +167,9 @@ typedef std::vector<OGroupArr::iterator> OActiveGroups;
 
 class OGroupManager : public ::cppu::WeakImplHelper2< 
::com::sun::star::beans::XPropertyChangeListener, 
::com::sun::star::container::XContainerListener>
 {
-    OGroup*         m_pCompGroup;           // Alle Components nach TabIndices 
sortiert
-    OGroupArr       m_aGroupArr;            // Alle Components nach Gruppen 
sortiert
-    OActiveGroups   m_aActiveGroupMap;      // In dieser Map werden die 
Indices aller Gruppen gehalten,
-                                        // die mehr als 1 Element haben
+    OGroup*         m_pCompGroup;           // Sort all Components by 
TabIndices
+    OGroupArr       m_aGroupArr;            // Sort all Components by group
+    OActiveGroups   m_aActiveGroupMap;      // This map contains all indices 
of all groups with more than 1 element
 
     ::com::sun::star::uno::Reference< ::com::sun::star::container::XContainer >
                     m_xContainer;
diff --git a/forms/source/component/Hidden.cxx 
b/forms/source/component/Hidden.cxx
index 16236a2..deb58ce 100644
--- a/forms/source/component/Hidden.cxx
+++ b/forms/source/component/Hidden.cxx
@@ -146,7 +146,7 @@ void SAL_CALL OHiddenModel::write(const 
Reference<XObjectOutputStream>& _rxOutSt
     // Version
     _rxOutStream->writeShort(0x0002);
 
-    // Wert
+    // Value
     _rxOutStream << m_sHiddenValue;
 
     OControlModel::write(_rxOutStream);
diff --git a/forms/source/component/ImageButton.cxx 
b/forms/source/component/ImageButton.cxx
index 868c41d..1802437 100644
--- a/forms/source/component/ImageButton.cxx
+++ b/forms/source/component/ImageButton.cxx
@@ -47,7 +47,7 @@ InterfaceRef SAL_CALL OImageButtonModel_CreateInstance(const 
Reference<XMultiSer
 
 OImageButtonModel::OImageButtonModel(const Reference<XComponentContext>& 
_rxFactory)
                     :OClickableImageBaseModel( _rxFactory, 
VCL_CONTROLMODEL_IMAGEBUTTON, FRM_SUN_CONTROL_IMAGEBUTTON )
-                                    // use the old control name for 
compytibility reasons
+                    // use the old control name for compytibility reasons
 {
     m_nClassId = FormComponentType::IMAGEBUTTON;
 }
@@ -173,7 +173,7 @@ OImageButtonControl::OImageButtonControl(const 
Reference<XComponentContext>& _rx
 {
     increment(m_refCount);
     {
-        // als MouseListener anmelden
+        // Register as MouseListener
         Reference< awt::XWindow >  xComp;
         query_aggregation( m_xAggregate, xComp);
         if (xComp.is())
@@ -182,7 +182,7 @@ OImageButtonControl::OImageButtonControl(const 
Reference<XComponentContext>& _rx
     decrement(m_refCount);
 }
 
-// UNO Anbindung
+// UNO Binding
 Any SAL_CALL OImageButtonControl::queryAggregation(const Type& _rType) throw 
(RuntimeException, std::exception)
 {
     Any aReturn = OClickableImageBaseControl::queryAggregation(_rType);
@@ -208,8 +208,8 @@ void OImageButtonControl::mousePressed(const 
awt::MouseEvent& e) throw ( ::com::
     }
     else
     {
-        // Sonst nicht. Dann darf man aber auf keinen Fal die Listener
-        // benachrichtigen, auch dann nicht, wenn er spaeter hinzukommt.
+        // Or else don't; we must not notify the listeners in that case.
+        // Even not if its added later on.
         aGuard.clear();
         actionPerformed_Impl( false, e );
     }
diff --git a/forms/source/component/ImageButton.hxx 
b/forms/source/component/ImageButton.hxx
index 645bb88..ee728e81 100644
--- a/forms/source/component/ImageButton.hxx
+++ b/forms/source/component/ImageButton.hxx
@@ -56,7 +56,7 @@ class OImageButtonControl : public OClickableImageBaseControl,
                             public OImageButtonControl_BASE
 {
 protected:
-    // UNO Anbindung
+    // UNO Binding
     virtual ::com::sun::star::uno::Sequence< ::com::sun::star::uno::Type> 
_getTypes() SAL_OVERRIDE;
 
 public:
@@ -66,7 +66,7 @@ public:
     IMPLEMENTATION_NAME(OImageButtonControl);
     virtual StringSequence SAL_CALL getSupportedServiceNames() 
throw(std::exception) SAL_OVERRIDE;
 
-    // UNO Anbindung
+    // UNO Binding
     DECLARE_UNO3_AGG_DEFAULTS(OImageButtonControl, OClickableImageBaseControl)
     virtual ::com::sun::star::uno::Any SAL_CALL queryAggregation(const 
::com::sun::star::uno::Type& _rType) 
throw(::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to