]> git.saurik.com Git - wxWidgets.git/blobdiff - interface/wx/propgrid/property.h
more fixes
[wxWidgets.git] / interface / wx / propgrid / property.h
index 65e494b41b9954b3306d42498b036536d931c95a..cd267eb3fd3f593737266b596cf9266916390262 100644 (file)
 #define wxNullProperty  ((wxPGProperty*)NULL)
 
 
-/** wxPGPaintData
-
-    Contains information relayed to property's OnCustomPaint.
-*/
-struct wxPGPaintData
-{
-    /** wxPropertyGrid. */
-    const wxPropertyGrid*   m_parent;
-
-    /** Normally -1, otherwise index to drop-down list item that has to be drawn. */
-    int                     m_choiceItem;
-
-    /** Set to drawn width in OnCustomPaint (optional). */
-    int                     m_drawnWidth;
-
-    /** In a measure item call, set this to the height of item at m_choiceItem index. */
-    int                     m_drawnHeight;   
-};
-
-
-// Structure for relaying choice/list info.
-struct wxPGChoiceInfo
-{
-    wxPGChoices*    m_choices;
-};
-
-
 /** @section propgrid_property_attributes wxPropertyGrid Property Attribute Identifiers
 
-    wxPGProperty::SetAttribute() and wxPropertyGridInterfaces::SetPropertyAttribute()
+    wxPGProperty::SetAttribute() and
+    wxPropertyGridInterface::SetPropertyAttribute()
     accept one of these as attribute name argument .
 
     You can use strings instead of constants. However, some of these
@@ -164,11 +138,6 @@ struct wxPGChoiceInfo
 */
 #define wxPG_COLOUR_ALLOW_CUSTOM            wxS("AllowCustom")
 
-/** First attribute id that is guaranteed not to be used built-in
-    properties.
-*/
-//#define wxPG_USER_ATTRIBUTE                 192
-
 /** @}
 */
 
@@ -218,16 +187,24 @@ struct wxPGChoiceInfo
     Simple string property. wxPG_STRING_PASSWORD attribute may be used
     to echo value as asterisks and use wxTE_PASSWORD for wxTextCtrl.
 
-    @remarks
-    * wxStringProperty has a special trait: if it has value of "<composed>",
-      and also has child properties, then its displayed value becomes
-      composition of child property values, similar as with wxFontProperty,
-      for instance.
+    @remarks wxStringProperty has a special trait: if it has value of
+            "<composed>", and also has child properties, then its displayed
+            value becomes composition of child property values, similar as
+            with wxFontProperty, for instance.
 
     @subsection wxIntProperty
 
     Like wxStringProperty, but converts text to a signed long integer.
     wxIntProperty seamlessly supports 64-bit integers (ie. wxLongLong).
+    To safely convert variant to integer, use code like this:
+
+    @code
+        wxLongLong ll;
+        ll << property->GetValue();
+
+        // or
+        wxLongLong ll = propertyGrid->GetPropertyValueAsLong(property);
+    @endcode
 
     @subsection wxUIntProperty
 
@@ -236,7 +213,8 @@ struct wxPGChoiceInfo
     To set the globally used base, manipulate wxPG_UINT_BASE int
     attribute. Regardless of current prefix, understands (hex) values starting
     with both "0x" and "$".
-    wxUIntProperty seamlessly supports 64-bit unsigned integers (ie. wxULongLong).
+    Like wxIntProperty, wxUIntProperty seamlessly supports 64-bit unsigned
+    integers (ie. wxULongLong). Same wxVariant safety rules apply.
 
     @subsection wxFloatProperty
 
@@ -281,7 +259,8 @@ struct wxPGChoiceInfo
     the flags as a text; a continous sequence of spaces, commas and semicolons
     is considered as a flag id separator.
     <b>Note: </b> When changing "choices" (ie. flag labels) of wxFlagsProperty, you
-    will need to use SetPropertyChoices - otherwise they will not get updated properly.
+    will need to use wxPGProperty::SetChoices() - otherwise they will not get updated
+    properly.
 
     @subsection wxArrayStringProperty
 
@@ -406,18 +385,22 @@ struct wxPGChoiceInfo
     they are only really needed if you need to use wxRTTI with your
     property class.
 
-    You can change the 'value type' of a property by simply assigning different type
-    of variant with SetValue. <b>It is mandatory to implement wxVariantData class
-    for all data types used as property values.</b> Also, it is further recommended
-    to derive your class from wxPGVariantData, like this:
+    You can change the 'value type' of a property by simply assigning different
+    type of variant with SetValue. <b>It is mandatory to implement
+    wxVariantData class for all data types used as property values.</b>
+    You can use macros declared in wxPropertyGrid headers. For instance:
 
     @code
         // In header file:
-        // (replace DECL with required data declaration, wxEMPTY_PARAMETER_VALUE if none)
-        WX_PG_DECLARE_VARIANT_DATA(wxPGVariantMyDataClass, MyDataClass, DECL)
+        // (If you need to have export declaration, use version of macros
+        // with _EXPORTED postfix)
+        WX_PG_DECLARE_VARIANT_DATA(MyDataClass)
 
         // In sources file:
-        WX_PG_IMPLEMENT_VARIANT_DATA(wxPGVariantMyDataClass, MyDataClass)
+        WX_PG_IMPLEMENT_VARIANT_DATA(MyDataClass)
+
+        // Or, if you don't have valid == operator:
+        WX_PG_IMPLEMENT_VARIANT_DATA_DUMMY_EQ(MyDataClass)
     @endcode
 
     @library{wxpropgrid}
@@ -428,18 +411,18 @@ class wxPGProperty : public wxObject
 public:
     typedef wxUint32 FlagType;
 
-    /** Basic constructor.
+    /**
+        Default constructor.
     */
     wxPGProperty();
 
-    /** Constructor.
+    /**
+        Constructor.
         Non-abstract property classes should have constructor of this style:
 
         @code
 
-        // If T is a class, then it should be a constant reference
-        // (e.g. const T& ) instead.
-        MyProperty( const wxString& label, const wxString& name, T value )
+        MyProperty( const wxString& label, const wxString& name, const T& value )
             : wxPGProperty()
         {
             // Generally recommended way to set the initial value
@@ -456,10 +439,13 @@ public:
     */
     wxPGProperty( const wxString& label, const wxString& name );
 
-    /** Virtual destructor. It is customary for derived properties to implement this. */
+    /**
+        Virtual destructor. It is customary for derived properties to implement this.
+    */
     virtual ~wxPGProperty();
 
-    /** This virtual function is called after m_value has been set.
+    /**
+        This virtual function is called after m_value has been set.
 
         @remarks
         - If m_value was set to Null variant (ie. unspecified value), OnSetValue()
@@ -471,11 +457,13 @@ public:
     */
     virtual void OnSetValue();
 
-    /** Override this to return something else than m_value as the value.
+    /**
+        Override this to return something else than m_value as the value.
     */
     virtual wxVariant DoGetValue() const { return m_value; }
 
-    /** Implement this function in derived class to check the value.
+    /**
+        Implement this function in derived class to check the value.
         Return true if it is ok. Returning false prevents property change events
         from occurring.
 
@@ -484,7 +472,8 @@ public:
     */
     virtual bool ValidateValue( wxVariant& value, wxPGValidationInfo& validationInfo ) const;
 
-    /** Converts 'text' into proper value 'variant'. Returns true if new (different than
+    /**
+        Converts 'text' into proper value 'variant'. Returns true if new (different than
         m_value) value could be interpreted from the text.
         @param argFlags
         If wxPG_FULL_VALUE is set, returns complete, storable value instead of displayable
@@ -499,7 +488,8 @@ public:
     */
     virtual bool StringToValue( wxVariant& variant, const wxString& text, int argFlags = 0 ) const;
 
-    /** Converts 'number' (including choice selection) into proper value 'variant'.
+    /**
+        Converts 'number' (including choice selection) into proper value 'variant'.
         Returns true if new (different than m_value) value could be interpreted from the integer.
         @param argFlags
         If wxPG_FULL_VALUE is set, returns complete, storable value instead of displayable one.
@@ -513,9 +503,8 @@ public:
     */
     virtual bool IntToValue( wxVariant& value, int number, int argFlags = 0 ) const;
 
-public:
-
-    /** Returns text representation of property's value.
+    /**
+        Returns text representation of property's value.
 
         @param argFlags
         If wxPG_FULL_VALUE is set, returns complete, storable string value instead of displayable.
@@ -529,7 +518,8 @@ public:
     */
     virtual wxString GetValueAsString( int argFlags = 0 ) const;
 
-    /** Converts string to a value, and if successful, calls SetValue() on it.
+    /**
+        Converts string to a value, and if successful, calls SetValue() on it.
         Default behavior is to do nothing.
         @param text
         String to get the value from.
@@ -538,7 +528,8 @@ public:
     */
     bool SetValueFromString( const wxString& text, int flags = 0 );
 
-    /** Converts integer to a value, and if succesful, calls SetValue() on it.
+    /**
+        Converts integer to a value, and if succesful, calls SetValue() on it.
         Default behavior is to do nothing.
         @param value
         Int to get the value from.
@@ -549,7 +540,8 @@ public:
     */
     bool SetValueFromInt( long value, int flags = 0 );
 
-    /** Returns size of the custom painted image in front of property. This method
+    /**
+        Returns size of the custom painted image in front of property. This method
         must be overridden to return non-default value if OnCustomPaint is to be
         called.
         @param item
@@ -561,7 +553,8 @@ public:
     */
     virtual wxSize OnMeasureImage( int item = -1 ) const;
 
-    /** Events received by editor widgets are processed here. Note that editor class
+    /**
+        Events received by editor widgets are processed here. Note that editor class
         usually processes most events. Some, such as button press events of
         TextCtrlAndButton class, can be handled here. Also, if custom handling
         for regular events is desired, then that can also be done (for example,
@@ -581,7 +574,8 @@ public:
     */
     virtual bool OnEvent( wxPropertyGrid* propgrid, wxWindow* wnd_primary, wxEvent& event );
 
-    /** Called after value of a child property has been altered. Note that this function is
+    /**
+        Called after value of a child property has been altered. Note that this function is
         usually called at the time that value of this property, or given child property, is
         still pending for change.
 
@@ -618,11 +612,13 @@ public:
     */
     virtual void ChildChanged( wxVariant& thisValue, int childIndex, wxVariant& childValue ) const;
 
-    /** Returns pointer to an instance of used editor.
+    /**
+        Returns pointer to an instance of used editor.
     */
     virtual const wxPGEditor* DoGetEditorClass() const;
 
-    /** Returns pointer to the wxValidator that should be used
+    /**
+        Returns pointer to the wxValidator that should be used
         with the editor of this property (NULL for no validator).
         Setting validator explicitly via SetPropertyValidator
         will override this.
@@ -654,18 +650,8 @@ public:
     */
     virtual wxValidator* DoGetValidator () const;
 
-    /** Returns current value's index to the choice control. May also return,
-        through pointer arguments, strings that should be inserted to that control.
-        Irrelevant to classes which do not employ wxPGEditor_Choice or similar.
-        @remarks
-        - If returns NULL in choiceinfo.m_choices, then this class must be
-          derived from wxBaseEnumProperty.
-        - Must be able to cope situation where property's set of choices is
-          uninitialized.
-    */
-    virtual int GetChoiceInfo( wxPGChoiceInfo* choiceinfo );
-
-    /** Override to paint an image in front of the property value text or drop-down
+    /**
+        Override to paint an image in front of the property value text or drop-down
         list item (but only if wxPGProperty::OnMeasureImage is overridden as well).
 
         If property's OnMeasureImage() returns size that has height != 0 but less than
@@ -688,7 +674,23 @@ public:
         Box reserved for custom graphics. Includes surrounding rectangle, if any.
         If x is < 0, then this is a measure item call (see above).
         @param paintdata
-        wxPGPaintData structure with much useful data.
+        wxPGPaintData structure with much useful data about painted item.
+        @code
+        struct wxPGPaintData
+        {
+            // wxPropertyGrid.
+            const wxPropertyGrid*   m_parent;
+
+            // Normally -1, otherwise index to drop-down list item that has to be drawn.
+            int                     m_choiceItem;
+
+            // Set to drawn width in OnCustomPaint (optional).
+            int                     m_drawnWidth;
+
+            // In a measure item call, set this to the height of item at m_choiceItem index
+            int                     m_drawnHeight;
+        };
+        @endcode
 
         @remarks
             - You can actually exceed rect width, but if you do so then paintdata.m_drawnWidth
@@ -704,17 +706,29 @@ public:
     */
     virtual void OnCustomPaint( wxDC& dc, const wxRect& rect, wxPGPaintData& paintdata );
 
-    /** Returns used wxPGCellRenderer instance for given property column (label=0, value=1).
+    /**
+        Returns used wxPGCellRenderer instance for given property column (label=0, value=1).
 
         Default implementation returns editor's renderer for all columns.
     */
     virtual wxPGCellRenderer* GetCellRenderer( int column ) const;
 
-    /** Refresh values of child properties. Automatically called after value is set.
+    /**
+        Returns which choice is currently selected. Only applies to properties
+        which have choices.
+
+        Needs to reimplemented in derived class if property value does not
+        map directly to a choice. Integer as index, bool, and string usually do.
+    */
+    virtual int GetChoiceSelection() const;
+
+    /**
+        Refresh values of child properties. Automatically called after value is set.
     */
     virtual void RefreshChildren();
 
-    /** Special handling for attributes of this property.
+    /**
+        Special handling for attributes of this property.
 
         If returns false, then the attribute will be automatically stored in
         m_attributes.
@@ -723,7 +737,8 @@ public:
     */
     virtual bool DoSetAttribute( const wxString& name, wxVariant& value );
 
-    /** Returns value of an attribute.
+    /**
+        Returns value of an attribute.
 
         Override if custom handling of attributes is needed.
 
@@ -731,7 +746,8 @@ public:
     */
     virtual wxVariant DoGetAttribute( const wxString& name ) const;
 
-    /** Returns instance of a new wxPGEditorDialogAdapter instance, which is
+    /**
+        Returns instance of a new wxPGEditorDialogAdapter instance, which is
         used when user presses the (optional) button next to the editor control;
 
         Default implementation returns NULL (ie. no action is generated when
@@ -739,15 +755,8 @@ public:
     */
     virtual wxPGEditorDialogAdapter* GetEditorDialog() const;
 
-    /** Adds entry to property's wxPGChoices and editor control (if it is active).
-        Returns index of item added.
-    */
-    int AppendChoice( const wxString& label, int value = wxPG_INVALID_VALUE )
-    {
-        return InsertChoice(label,-1,value);
-    }
-
-    /** Returns wxPGCell of given column, NULL if none. If valid
+    /**
+        Returns wxPGCell of given column, NULL if none. If valid
         object is returned, caller will gain its ownership.
     */
     wxPGCell* AcquireCell( unsigned int column )
@@ -760,7 +769,44 @@ public:
         return cell;
     }
 
-    /** Returns true if children of this property are component values (for instance,
+    /**
+        Append a new choice to property's list of choices.
+
+        @param label
+            Label for added choice.
+
+        @param value
+            Value for new choice. Do not specify if you wish this
+            to equal choice index.
+
+        @return
+            Index to added choice.
+    */
+    int AddChoice( const wxString& label, int value = wxPG_INVALID_VALUE );
+
+    /**
+        Properties which have private child properties should add them
+        with this function, called in their constructor.
+    */
+    void AddChild( wxPGProperty* property );
+
+    /**
+        Adapts list variant into proper value using consecutive
+        ChildChanged() calls.
+    */
+    void AdaptListToValue( wxVariant& list, wxVariant* value ) const;
+
+    /**
+        Determines, recursively, if all children are not unspecified.
+
+        @param pendingList
+            Assumes members in this wxVariant list as pending
+            replacement values.
+    */
+    bool AreAllChildrenSpecified( wxVariant* pendingList = NULL ) const;
+
+    /**
+        Returns true if children of this property are component values (for instance,
         points size, face name, and is_underlined are component values of a font).
     */
     bool AreChildrenComponents() const
@@ -771,72 +817,57 @@ public:
         return false;
     }
 
-    /** Removes entry from property's wxPGChoices and editor control (if it is active).
+    /**
+        Removes entry from property's wxPGChoices and editor control (if it is active).
 
         If selected item is deleted, then the value is set to unspecified.
     */
     void DeleteChoice( int index );
 
-    /** Call to enable or disable usage of common value (integer value that can be selected for
-        properties instead of their normal values) for this property.
-
-        Common values are disabled by the default for all properties.
-    */
-    void EnableCommonValue( bool enable = true )
-    {
-        if ( enable ) SetFlag( wxPG_PROP_USES_COMMON_VALUE );
-        else ClearFlag( wxPG_PROP_USES_COMMON_VALUE );
-    }
+    /** Deletes all child properties. */
+    void Empty();
 
     /** Composes text from values of child properties. */
     void GenerateComposedValue( wxString& text, int argFlags = 0 ) const;
 
-    /** Returns property's label. */
-    const wxString& GetLabel() const { return m_label; }
-
-    /** Returns property's name with all (non-category, non-root) parents. */
-    wxString GetName() const;
-
-    /** Returns property's base name (ie. parent's name is not added in any case) */
-    const wxString& GetBaseName() const { return m_name; }
-
-    wxPGChoices& GetChoices();
+    /**
+        Returns property attribute value, null variant if not found.
+    */
+    wxVariant GetAttribute( const wxString& name ) const;
 
-    const wxPGChoices& GetChoices() const;
+    /** Returns named attribute, as string, if found. Otherwise defVal is returned.
+    */
+    wxString GetAttribute( const wxString& name, const wxString& defVal ) const;
 
-    const wxPGChoiceEntry* GetCurrentChoice() const;
+    /** Returns named attribute, as long, if found. Otherwise defVal is returned.
+    */
+    long GetAttributeAsLong( const wxString& name, long defVal ) const;
 
-    /** Returns coordinate to the top y of the property. Note that the
-        position of scrollbars is not taken into account.
+    /** Returns named attribute, as double, if found. Otherwise defVal is returned.
     */
-    int GetY() const;
+    double GetAttributeAsDouble( const wxString& name, double defVal ) const;
 
-    wxVariant GetValue() const
-    {
-        return DoGetValue();
-    }
+    /**
+        Returns attributes as list wxVariant.
+    */
+    wxVariant GetAttributesAsList() const;
 
-    /** Returns reference to the internal stored value. GetValue is preferred
-        way to get the actual value, since GetValueRef ignores DoGetValue,
-        which may override stored value.
+    /**
+        Returns editor used for given column. NULL for no editor.
     */
-    wxVariant& GetValueRef()
+    const wxPGEditor* GetColumnEditor( int column ) const
     {
-        return m_value;
-    }
+        if ( column == 1 )
+            return GetEditorClass();
 
-    const wxVariant& GetValueRef() const
-    {
-        return m_value;
+        return NULL;
     }
 
-    /** Same as GetValueAsString, except takes common value into account.
-    */
-    wxString GetValueString( int argFlags = 0 ) const;
-
-    void UpdateControl( wxWindow* primary );
+    /** Returns property's base name (ie. parent's name is not added in any case) */
+    const wxString& GetBaseName() const { return m_name; }
 
-    /** Returns wxPGCell of given column, NULL if none. wxPGProperty
+    /**
+        Returns wxPGCell of given column, NULL if none. wxPGProperty
         will retain ownership of the cell object.
     */
     wxPGCell* GetCell( unsigned int column ) const
@@ -847,472 +878,422 @@ public:
         return (wxPGCell*) m_cells[column];
     }
 
-    unsigned int GetChoiceCount() const;
+    /**
+        Returns number of child properties.
+    */
+    unsigned int GetChildCount() const;
+
+    /**
+        Returns height of children, recursively, and
+        by taking expanded/collapsed status into account.
+
+        @param lh
+            Line height. Pass result of GetGrid()->GetRowHeight() here.
+
+        @param iMax
+            Only used (internally) when finding property y-positions.
+    */
+    int GetChildrenHeight( int lh, int iMax = -1 ) const;
+
+    /**
+        Returns read-only reference to property's list of choices.
+    */
+    const wxPGChoices& GetChoices() const;
+
+    /**
+        Returns client data (void*) of a property.
+    */
+    void* GetClientData() const;
 
-    wxString GetChoiceString( unsigned int index );
+    /** Sets managed client object of a property.
+    */
+    wxClientData *GetClientObject() const;
 
-    /** Return number of displayed common values for this property.
+    /**
+        Returns property's default value. If property's value type is not
+        a built-in one, and "DefaultValue" attribute is not defined, then
+        this function usually returns Null variant.
     */
-    int GetDisplayedCommonValueCount() const;
+    wxVariant GetDefaultValue() const;
 
+    /** Returns property's displayed text.
+    */
     wxString GetDisplayedString() const
     {
         return GetValueString(0);
     }
 
+    /**
+        Returns wxPGEditor that will be used and created when
+        property becomes selected. Returns more accurate value
+        than DoGetEditorClass().
+    */
+    const wxPGEditor* GetEditorClass() const;
+
     /** Returns property grid where property lies. */
     wxPropertyGrid* GetGrid() const;
 
-    /** Returns owner wxPropertyGrid, but only if one is currently on a page
-        displaying this property. */
+    /**
+        Returns owner wxPropertyGrid, but only if one is currently on a page
+        displaying this property.
+    */
     wxPropertyGrid* GetGridIfDisplayed() const;
 
-    /** Returns highest level non-category, non-root parent. Useful when you
-        have nested wxCustomProperties/wxParentProperties.
-        @remarks
-        Thus, if immediate parent is root or category, this will return the
-        property itself.
+    /**
+        Returns property's help or description text.
+
+        @see SetHelpString()
     */
-    wxPGProperty* GetMainParent() const;
+    const wxString& GetHelpString() const;
 
-    /** Return parent of property */
-    wxPGProperty* GetParent() const { return m_parent; }
+    /**
+        Returns position in parent's array.
+    */
+    unsigned int GetIndexInParent() const;
 
-    /** Returns true if property has editable wxTextCtrl when selected.
+    /** Returns property's label. */
+    const wxString& GetLabel() const { return m_label; }
 
-        @remarks
-        Altough disabled properties do not displayed editor, they still
-        return True here as being disabled is considered a temporary
-        condition (unlike being read-only or having limited editing enabled).
+    /**
+        Returns last visible child property, recursively.
     */
-    bool IsTextEditable() const;
-
-    bool IsValueUnspecified() const
-    {
-        return m_value.IsNull();
-    }
+    const wxPGProperty* GetLastVisibleSubItem() const;
 
-    FlagType HasFlag( FlagType flag ) const
-    {
-        return ( m_flags & flag );
-    }
+    /**
+        Returns highest level non-category, non-root parent. Useful when you
+        have nested wxCustomProperties/wxParentProperties.
 
-    /** Returns comma-delimited string of property attributes.
+        @remarks If immediate parent is root or category, this will return the
+                property itself.
     */
-    const wxPGAttributeStorage& GetAttributes() const
-    {
-        return m_attributes;
-    }
+    wxPGProperty* GetMainParent() const;
 
-    /** Returns m_attributes as list wxVariant.
+    /** Returns maximum allowed length of property's text value.
     */
-    wxVariant GetAttributesAsList() const;
-
-    FlagType GetFlags() const
+    int GetMaxLength() const
     {
-        return m_flags;
+        return (int) m_maxLen;
     }
 
-    const wxPGEditor* GetEditorClass() const;
+    /** Returns property's name with all (non-category, non-root) parents. */
+    wxString GetName() const;
 
-    wxString GetValueType() const
-    {
-        return m_value.GetType();
-    }
+    /** Return parent of property */
+    wxPGProperty* GetParent() const { return m_parent; }
 
-    /** Returns editor used for given column. NULL for no editor.
+    /**
+        Returns (direct) child property with given name (or NULL if not found).
     */
-    const wxPGEditor* GetColumnEditor( int column ) const
-    {
-        if ( column == 1 )
-            return GetEditorClass();
+    wxPGProperty* GetPropertyByName( const wxString& name ) const;
 
-        return NULL;
-    }
+    /** Gets assignable version of property's validator. */
+    wxValidator* GetValidator() const;
 
-    /** Returns common value selected for this property. -1 for none.
+    /**
+        Returns property's value.
     */
-    int GetCommonValue() const
+    wxVariant GetValue() const
     {
-        return m_commonValue;
+        return DoGetValue();
     }
 
-    /** Returns true if property has even one visible child.
+    /**
+        Returns bitmap that appears next to value text. Only returns non-NULL
+        bitmap if one was set with SetValueImage().
     */
-    bool HasVisibleChildren() const;
+    wxBitmap* GetValueImage() const;
 
-    /** Adds entry to property's wxPGChoices and editor control (if it is active).
-        Returns index of item added.
-    */
-    int InsertChoice( const wxString& label, int index, int value = wxPG_INVALID_VALUE );
+    /**
+        To acquire property's value as string, you should use this
+        function (instead of GetValueAsString()), as it may produce
+        more accurate value in future versions.
 
-    /** Returns true if this property is actually a wxPropertyCategory.
+        @see GetValueAsString()
     */
-    bool IsCategory() const { return HasFlag(wxPG_PROP_CATEGORY)?true:false; }
+    wxString GetValueString( int argFlags = 0 ) const;
 
-    /** Returns true if this property is actually a wxRootProperty.
+    /**
+        Returns value type used by this property.
     */
-    bool IsRoot() const { return (m_parent == NULL); }
-
-    /** Returns true if this is a sub-property. */
-    bool IsSubProperty() const
+    wxString GetValueType() const
     {
-        wxPGProperty* parent = (wxPGProperty*)m_parent;
-        if ( parent && !parent->IsCategory() )
-            return true;
-        return false;
+        return m_value.GetType();
     }
 
-    /** Returns last visible sub-property, recursively.
+    /**
+        Returns coordinate to the top y of the property. Note that the
+        position of scrollbars is not taken into account.
     */
-    const wxPGProperty* GetLastVisibleSubItem() const;
-
-    wxVariant GetDefaultValue() const;
-
-    int GetMaxLength() const
-    {
-        return (int) m_maxLen;
-    }
+    int GetY() const;
 
-    /** Determines, recursively, if all children are not unspecified. Takes values in given list into account.
+    /**
+        Returns true if property has even one visible child.
     */
-    bool AreAllChildrenSpecified( wxVariant* pendingList = NULL ) const;
+    bool HasVisibleChildren() const;
 
-    /** Updates composed values of parent non-category properties, recursively.
-        Returns topmost property updated.
+    /**
+        Hides or reveals the property.
 
-        @remarks
-        - Must not call SetValue() (as can be called in it).
+        @param hide
+            @true for hide, @false for reveal.
+
+        @param flags
+            By default changes are applied recursively. Set this parameter wxPG_DONT_RECURSE to prevent this.
     */
-    wxPGProperty* UpdateParentValues();
+    bool Hide( bool hide, int flags = wxPG_RECURSE );
 
-    /** Returns true if containing grid uses wxPG_EX_AUTO_UNSPECIFIED_VALUES.
+    /**
+        Returns index of given child property. wxNOT_FOUND if
+        given property is not child of this.
     */
-    FlagType UsesAutoUnspecified() const
-    {
-        return HasFlag(wxPG_PROP_AUTO_UNSPECIFIED);
-    }
+    int Index( const wxPGProperty* p ) const;
 
-    wxBitmap* GetValueImage() const
-    {
-        return m_valueBitmap;
-    }
+    /**
+        Inserts a new choice to property's list of choices.
 
-    wxVariant GetAttribute( const wxString& name ) const;
+        @param label
+            Text for new choice
 
-    /** Returns named attribute, as string, if found. Otherwise defVal is returned.
+        @param index
+            Insertion position. Use wxNOT_FOUND to append.
+
+        @param value
+            Value for new choice. Do not specify if you wish this
+            to equal choice index.
     */
-    wxString GetAttribute( const wxString& name, const wxString& defVal ) const;
+    int InsertChoice( const wxString& label, int index, int value = wxPG_INVALID_VALUE );
 
-    /** Returns named attribute, as long, if found. Otherwise defVal is returned.
+    /**
+        Returns @true if this property is actually a wxPropertyCategory.
     */
-    long GetAttributeAsLong( const wxString& name, long defVal ) const;
+    bool IsCategory() const;
 
-    /** Returns named attribute, as double, if found. Otherwise defVal is returned.
+    /**
+        Returns @true if property is enabled.
     */
-    double GetAttributeAsDouble( const wxString& name, double defVal ) const;
+    bool IsEnabled() const;
 
-    inline unsigned int GetArrIndex() const { return m_arrIndex; }
+    /**
+        Returns @true if property has visible children.
+    */
+    bool IsExpanded() const;
 
-    inline unsigned int GetDepth() const { return (unsigned int)m_depth; }
+    /**
+        Returns @true if this property is actually a wxRootProperty.
+    */
+    bool IsRoot() const { return (m_parent == NULL); }
 
-    /** Gets flags as a'|' delimited string. Note that flag names are not
-        prepended with 'wxPG_PROP_'.
-        @param flagsMask
-        String will only be made to include flags combined by this parameter.
+    /**
+        Returns @true if candidateParent is some parent of this property.
     */
-    wxString GetFlagsAsString( FlagType flagsMask ) const;
+    bool IsSomeParent( wxPGProperty* candidateParent ) const;
 
-    /** Returns position in parent's array. */
-    unsigned int GetIndexInParent() const
-    {
-        return (unsigned int)m_arrIndex;
-    }
+    /**
+        Returns true if property has editable wxTextCtrl when selected.
 
-    /** Hides or reveals the property.
-        @param hide
-        true for hide, false for reveal.
-        @param flags
-        By default changes are applied recursively. Set this paramter wxPG_DONT_RECURSE to prevent this.
+        @remarks Altough disabled properties do not displayed editor, they still
+                return @true here as being disabled is considered a temporary
+                condition (unlike being read-only or having limited editing enabled).
     */
-    inline bool Hide( bool hide, int flags = wxPG_RECURSE );
+    bool IsTextEditable() const;
 
-    bool IsExpanded() const { return (!(m_flags & wxPG_PROP_COLLAPSED) && GetChildCount()); }
+    /**
+        Returns @true if property's value is considered unspecified. This
+        usually means that value is Null variant.
+    */
+    bool IsValueUnspecified() const;
 
-    /** Returns true if all parents expanded.
+    /**
+        Returns true if all parents expanded.
     */
     bool IsVisible() const;
 
-    bool IsEnabled() const
-    {
-        return ( m_flags & wxPG_PROP_DISABLED ) ? false : true;
-    }
+    /**
+        Returns child property at index i.
+    */
+    wxPGProperty* Item( size_t i ) const;
+
+    /**
+        Updates property value in case there were last minute
+        changes. If value was unspecified, it will be set to default.
+        Use only for properties that have TextCtrl-based editor.
+
+        @remarks If you have code similar to
+                @code
+                // Update the value in case of last minute changes
+                if ( primary && propgrid->IsEditorsValueModified() )
+                     GetEditorClass()->CopyValueFromControl( this, primary );
+                @endcode
+                in wxPGProperty::OnEvent wxEVT_COMMAND_BUTTON_CLICKED handler,
+                then replace it with call to this method.
 
-    /** If property's editor is created this forces its recreation. Useful
-        in SetAttribute etc. Returns true if actually did anything.
+        @return Returns @true if value changed.
     */
-    bool RecreateEditor();
+    bool PrepareValueForDialogEditing( wxPropertyGrid* propgrid );
 
-    /** If property's editor is active, then update it's value.
+    /**
+        If property's editor is active, then update it's value.
     */
     void RefreshEditor();
 
-    /** Sets an attribute for this property.
+    /**
+        Sets an attribute for this property.
+
         @param name
-        Text identifier of attribute. See @ref propgrid_property_attributes.
+            Text identifier of attribute. See @ref propgrid_property_attributes.
+
         @param value
-        Value of attribute.
+            Value of attribute.
+
+        @remarks Setting attribute's value to Null variant will simply remove it
+                from property's set of attributes.
     */
     void SetAttribute( const wxString& name, wxVariant value );
 
-    void SetAttributes( const wxPGAttributeStorage& attributes );
-
-    /** Sets editor for a property.
+    /**
+        Sets editor for a property.
 
         @param editor
-        For builtin editors, use wxPGEditor_X, where X is builtin editor's
-        name (TextCtrl, Choice, etc. see wxPGEditor documentation for full list).
+            For builtin editors, use wxPGEditor_X, where X is builtin editor's
+            name (TextCtrl, Choice, etc. see wxPGEditor documentation for full list).
 
         For custom editors, use pointer you received from wxPropertyGrid::RegisterEditorClass().
     */
-    void SetEditor( const wxPGEditor* editor )
-    {
-        m_customEditor = editor;
-    }
+    void SetEditor( const wxPGEditor* editor );
 
-    /** Sets editor for a property.
+    /**
+        Sets editor for a property, by editor name.
     */
-    inline void SetEditor( const wxString& editorName );
+    void SetEditor( const wxString& editorName );
 
-    /** Sets cell information for given column.
+    /**
+        Sets cell information for given column.
 
         Note that the property takes ownership of given wxPGCell instance.
     */
     void SetCell( int column, wxPGCell* cellObj );
 
-    /** Changes value of a property with choices, but only
-        works if the value type is long or string. */
-    void SetChoiceSelection( int newValue, const wxPGChoiceInfo& choiceInfo );
-
-    /** Sets common value selected for this property. -1 for none.
-    */
-    void SetCommonValue( int commonValue )
-    {
-        m_commonValue = commonValue;
-    }
+    /**
+        Sets new set of choices for property.
 
-    /** Sets flags from a '|' delimited string. Note that flag names are not
-        prepended with 'wxPG_PROP_'.
+        @remarks This operation clears the property value.
     */
-    void SetFlagsFromString( const wxString& str );
+    bool SetChoices( wxPGChoices& choices );
 
-    /** Sets property's "is it modified?" flag. Affects children recursively.
+    /**
+        If property has choices and they are not yet exclusive, new such copy
+        of them will be created.
     */
-    void SetModifiedStatus( bool modified )
-    {
-        SetFlagRecursively(wxPG_PROP_MODIFIED, modified);
-    }
+    void SetChoicesExclusive();
 
-    /** Call in OnEvent(), OnButtonClick() etc. to change the property value
-        based on user input.
+    /**
+        Sets client data (void*) of a property.
 
-        @remarks
-        This method is const since it doesn't actually modify value, but posts
-        given variant as pending value, stored in wxPropertyGrid.
+        @remarks This untyped client data has to be deleted manually.
     */
-    void SetValueInEvent( wxVariant value ) const;
+    void SetClientData( void* clientData );
 
-    /** Call this to set value of the property. Unlike methods in wxPropertyGrid,
-        this does not automatically update the display.
-
-        @remarks
-        Use wxPropertyGrid::ChangePropertyValue() instead if you need to run through
-        validation process and send property change event.
-
-        If you need to change property value in event, based on user input, use
-        SetValueInEvent() instead.
-
-        @param pList
-        Pointer to list variant that contains child values. Used to indicate
-        which children should be marked as modified.
-        @param flags
-        Various flags (for instance, wxPG_SETVAL_REFRESH_EDITOR).
+    /** Returns client object of a property.
     */
-    void SetValue( wxVariant value, wxVariant* pList = NULL, int flags = 0 );
+    void SetClientObject(wxClientData* clientObject);
 
-    /** Set wxBitmap in front of the value. This bitmap may be ignored
-        by custom cell renderers.
-    */
-    void SetValueImage( wxBitmap& bmp );
+    /**
+        Sets selected choice and changes property value.
 
-    /** If property has choices and they are not yet exclusive, new such copy
-        of them will be created.
+        Tries to retain value type, although currently if it is not string,
+        then it is forced to integer.
     */
-    void SetChoicesExclusive();
-
-    void SetExpanded( bool expanded )
-    {
-        if ( !expanded ) m_flags |= wxPG_PROP_COLLAPSED;
-        else m_flags &= ~wxPG_PROP_COLLAPSED;
-    }
-
-    void SetFlag( FlagType flag ) { m_flags |= flag; }
+    void SetChoiceSelection( int newValue );
 
-    void SetFlagRecursively( FlagType flag, bool set );
-
-    void SetHelpString( const wxString& helpString )
-    {
-        m_helpString = helpString;
-    }
-
-    /** Sets property's label.
-
-        @remarks
-        - Properties under same parent may have same labels. However,
-        property names must still remain unique.
+    /**
+        Sets property's help string, which is shown, for example, in
+        wxPropertyGridManager's description text box.
     */
-    void SetLabel( const wxString& label ) { m_label = label; }
-
-    inline void SetName( const wxString& newName );
-
-    void SetValueToUnspecified()
-    {
-        wxVariant val;  // Create NULL variant
-        SetValue(val);
-    }
-
-    /** Sets wxValidator for a property*/
-    void SetValidator( const wxValidator& validator )
-    {
-        m_validator = wxDynamicCast(validator.Clone(),wxValidator);
-    }
+    void SetHelpString( const wxString& helpString );
 
-    /** Gets assignable version of property's validator. */
-    wxValidator* GetValidator() const
-    {
-        if ( m_validator )
-            return m_validator;
-        return DoGetValidator();
-    }
+    /**
+        Sets property's label.
 
-    /** Updates property value in case there were last minute
-        changes. If value was unspecified, it will be set to default.
-        Use only for properties that have TextCtrl-based editor.
-        @remarks
-        If you have code similar to
-        @code
-            // Update the value in case of last minute changes
-            if ( primary && propgrid->IsEditorsValueModified() )
-                 GetEditorClass()->CopyValueFromControl( this, primary );
-        @endcode
-        in wxPGProperty::OnEvent wxEVT_COMMAND_BUTTON_CLICKED handler,
-        then replace it with call to this method.
-        @retval
-        True if value changed.
+        @remarks Properties under same parent may have same labels. However,
+                property names must still remain unique.
     */
-    bool PrepareValueForDialogEditing( wxPropertyGrid* propgrid );
+    void SetLabel( const wxString& label ) { m_label = label; }
 
-    /** Returns client data (void*) of a property.
+    /**
+        Set max length of text in text editor.
     */
-    void* GetClientData() const
-    {
-        return m_clientData;
-    }
+    bool SetMaxLength( int maxLen );
 
-    /** Sets client data (void*) of a property.
-        @remarks
-        This untyped client data has to be deleted manually.
+    /**
+        Sets property's "is it modified?" flag. Affects children recursively.
     */
-    void SetClientData( void* clientData )
-    {
-        m_clientData = clientData;
-    }
+    void SetModifiedStatus( bool modified );
 
-    /** Returns client object of a property.
+    /**
+        Sets new (base) name for property.
     */
-    void SetClientObject(wxClientData* clientObject)
-    {
-        delete m_clientObject;
-        m_clientObject = clientObject;
-    }
+    void SetName( const wxString& newName );
 
-    /** Sets managed client object of a property.
-    */
-    wxClientData *GetClientObject() const { return m_clientObject; }
+    /** Sets wxValidator for a property */
+    void SetValidator( const wxValidator& validator );
 
-    /** Sets new set of choices for property.
+    /**
+        Call this to set value of the property. Unlike methods in wxPropertyGrid,
+        this does not automatically update the display.
 
         @remarks
-        This operation clears the property value.
-    */
-    bool SetChoices( wxPGChoices& choices );
+            Use wxPropertyGrid::ChangePropertyValue() instead if you need to run through
+            validation process and send property change event.
 
-    /** Sets new set of choices for property.
-    */
-    inline bool SetChoices( const wxArrayString& labels,
-                            const wxArrayInt& values = wxArrayInt() );
+            If you need to change property value in event, based on user input, use
+            SetValueInEvent() instead.
 
-    /** Set max length of text in text editor.
-    */
-    inline bool SetMaxLength( int maxLen );
+        @param pList
+            Pointer to list variant that contains child values. Used to indicate
+            which children should be marked as modified. Usually you just use NULL.
 
-    /** Call with 'false' in OnSetValue to cancel value changes after all
-        (ie. cancel 'true' returned by StringToValue() or IntToValue()).
+        @param flags
+            Use wxPG_SETVAL_REFRESH_EDITOR to update editor control, if it
+            was selected.
     */
-    void SetWasModified( bool set = true )
-    {
-        if ( set ) m_flags |= wxPG_PROP_WAS_MODIFIED;
-        else m_flags &= ~wxPG_PROP_WAS_MODIFIED;
-    }
-
-    const wxString& GetHelpString() const
-    {
-        return m_helpString;
-    }
-
-    void ClearFlag( FlagType flag ) { m_flags &= ~(flag); }
-
-    // Use, for example, to detect if item is inside collapsed section.
-    bool IsSomeParent( wxPGProperty* candidate_parent ) const;
+    void SetValue( wxVariant value, wxVariant* pList = NULL, int flags = 0 );
 
-    /** Adapts list variant into proper value using consequtive ChildChanged-calls.
+    /**
+        Set wxBitmap in front of the value. This bitmap may be ignored
+        by custom cell renderers.
     */
-    void AdaptListToValue( wxVariant& list, wxVariant* value ) const;
+    void SetValueImage( wxBitmap& bmp );
 
-    /** This is used by properties that have fixed sub-properties. */
-    void AddChild( wxPGProperty* prop );
+    /**
+        Call this function in OnEvent(), OnButtonClick() etc. to change the
+        property value based on user input.
 
-    /** Returns height of children, recursively, and
-        by taking expanded/collapsed status into account.
-        
-        iMax is used when finding property y-positions.
+        @remarks This method is const since it doesn't actually modify value, but posts
+                given variant as pending value, stored in wxPropertyGrid.
     */
-    int GetChildrenHeight( int lh, int iMax = -1 ) const;
-
-    /** Returns number of child properties */
-    unsigned int GetChildCount() const { return m_children.GetCount(); }
-
-    /** Returns sub-property at index i. */
-    wxPGProperty* Item( size_t i ) const { return (wxPGProperty*)m_children.Item(i); }
+    void SetValueInEvent( wxVariant value ) const;
 
-    /** Returns last sub-property.
+    /**
+        Sets property's value to unspecified (ie. Null variant).
     */
-    wxPGProperty* Last() const { return (wxPGProperty*)m_children.Last(); }
-
-    /** Returns index of given sub-property. */
-    int Index( const wxPGProperty* p ) const { return m_children.Index((wxPGProperty*)p); }
-
-    /** Deletes all sub-properties. */
-    void Empty();
+    void SetValueToUnspecified();
 
-    // Puts correct indexes to children
-    void FixIndexesOfChildren( size_t starthere = 0 );
+    /**
+        Call with @false in OnSetValue() to cancel value changes after all
+        (ie. cancel @true returned by StringToValue() or IntToValue()).
+    */
+    void SetWasModified( bool set = true );
 
-    wxPGProperty* GetItemAtY( unsigned int y, unsigned int lh, unsigned int* nextItemY ) const;
+    /**
+        Updates composed values of parent non-category properties, recursively.
+        Returns topmost property updated.
+    */
+    wxPGProperty* UpdateParentValues();
 
-    /** Returns (direct) child property with given name (or NULL if not found).
+    /**
+        Returns true if containing grid uses wxPG_EX_AUTO_UNSPECIFIED_VALUES.
     */
-    wxPGProperty* GetPropertyByName( const wxString& name ) const;
+    bool UsesAutoUnspecified() const;
 };
 
 
@@ -1329,51 +1310,28 @@ class WXDLLIMPEXP_PROPGRID wxPGChoices
 public:
     typedef long ValArrItem;
 
-    /** Default constructor. */
-    wxPGChoices()
-    {
-        Init();
-    }
+    /**
+        Default constructor.
+    */
+    wxPGChoices();
 
     /** Copy constructor. */
-    wxPGChoices( const wxPGChoices& a )
-    {
-        if ( a.m_data != wxPGChoicesEmptyData )
-        {
-            m_data = a.m_data;
-            m_data->m_refCount++;
-        }
-    }
+    wxPGChoices( const wxPGChoices& a );
 
     /** Constructor. */
-    wxPGChoices( const wxChar** labels, const long* values = NULL )
-    {
-        Init();
-        Set(labels,values);
-    }
+    wxPGChoices( const wxChar** labels, const long* values = NULL );
 
     /** Constructor. */
-    wxPGChoices( const wxArrayString& labels, const wxArrayInt& values = wxArrayInt() )
-    {
-        Init();
-        Set(labels,values);
-    }
+    wxPGChoices( const wxArrayString& labels, const wxArrayInt& values = wxArrayInt() );
 
-    /** Simple interface constructor. */
-    wxPGChoices( wxPGChoicesData* data )
-    {
-        wxASSERT(data);
-        m_data = data;
-        data->m_refCount++;
-    }
+    /** Constructor. */
+    wxPGChoices( wxPGChoicesData* data );
 
     /** Destructor. */
-    ~wxPGChoices()
-    {
-        Free();
-    }
+    ~wxPGChoices();
 
-    /** Adds to current. If did not have own copies, creates them now. If was empty,
+    /**
+        Adds to current. If did not have own copies, creates them now. If was empty,
         identical to set except that creates copies.
     */
     void Add( const wxChar** labels, const ValArrItem* values = NULL );
@@ -1388,7 +1346,8 @@ public:
     wxPGChoiceEntry& Add( const wxString& label, int value = wxPG_INVALID_VALUE );
 
     /** Adds a single item, with bitmap. */
-    wxPGChoiceEntry& Add( const wxString& label, const wxBitmap& bitmap, int value = wxPG_INVALID_VALUE );
+    wxPGChoiceEntry& Add( const wxString& label, const wxBitmap& bitmap,
+                          int value = wxPG_INVALID_VALUE );
 
     /** Adds a single item with full entry information. */
     wxPGChoiceEntry& Add( const wxPGChoiceEntry& entry )
@@ -1396,74 +1355,86 @@ public:
         return Insert(entry, -1);
     }
 
-    /** Adds single item. */
+    /** Adds single item, sorted. */
     wxPGChoiceEntry& AddAsSorted( const wxString& label, int value = wxPG_INVALID_VALUE );
 
+    /**
+        Assigns data from another set of choices.
+    */
     void Assign( const wxPGChoices& a )
     {
         AssignData(a.m_data);
     }
 
+    /**
+        Assigns data from another set of choices.
+    */
     void AssignData( wxPGChoicesData* data );
 
-    /** Delete all choices. */
+    /**
+        Deletes all items.
+    */
     void Clear()
     {
         if ( m_data != wxPGChoicesEmptyData )
             m_data->Clear();
     }
 
-    void EnsureData()
-    {
-        if ( m_data == wxPGChoicesEmptyData )
-            m_data = new wxPGChoicesData();
-    }
-
-    /** Gets a unsigned number identifying this list. */
-    wxPGChoicesId GetId() const { return (wxPGChoicesId) m_data; };
-
-    const wxString& GetLabel( size_t ind ) const
-    {
-        wxASSERT( ind >= 0 && ind < GetCount() );
-        return Item(ind).GetText();
-    }
+    /**
+        Returns labe of item.
+    */
+    const wxString& GetLabel( size_t ind ) const;
 
-    size_t GetCount () const
-    {
-        wxASSERT_MSG( m_data,
-            wxT("When checking if wxPGChoices is valid, use IsOk() instead of GetCount()") );
-        return m_data->GetCount();
-    }
+    /**
+        Returns number of items.
+    */
+    size_t GetCount () const;
 
-    int GetValue( size_t ind ) const { return Item(ind).GetValue(); }
+    /**
+        Returns value of item;
+    */
+    int GetValue( size_t ind ) const;
 
-    /** Returns array of values matching the given strings. Unmatching strings
+    /**
+        Returns array of values matching the given strings. Unmatching strings
         result in wxPG_INVALID_VALUE entry in array.
     */
     wxArrayInt GetValuesForStrings( const wxArrayString& strings ) const;
 
-    /** Returns array of indices matching given strings. Unmatching strings
+    /**
+        Returns array of indices matching given strings. Unmatching strings
         are added to 'unmatched', if not NULL.
     */
-    wxArrayInt GetIndicesForStrings( const wxArrayString& strings, wxArrayString* unmatched = NULL ) const;
+    wxArrayInt GetIndicesForStrings( const wxArrayString& strings,
+                                     wxArrayString* unmatched = NULL ) const;
 
-    /** Returns true if choices in general are likely to have values
-        (depens on that all entries have values or none has)
+    /**
+        Returns @true if item at given index has a valid value;
     */
-    bool HasValues() const;
+    bool HasValue( unsigned int i ) const;
 
-    bool HasValue( unsigned int i ) const { return (m_data->GetCount() > i && m_data->Item(i)->HasValue()); }
+    /**
+        Returns index of item with given label.
+    */
+    int Index( const wxString& label ) const;
 
-    int Index( const wxString& str ) const;
+    /**
+        Returns index of item with given value.
+    */
     int Index( int val ) const;
 
-    /** Inserts single item. */
+    /**
+        Inserts single item.
+    */
     wxPGChoiceEntry& Insert( const wxString& label, int index, int value = wxPG_INVALID_VALUE );
 
-    /** Inserts a single item with full entry information. */
+    /**
+        Inserts a single item with full entry information.
+    */
     wxPGChoiceEntry& Insert( const wxPGChoiceEntry& entry, int index );
 
-    /** Returns false if this is a constant empty set of choices,
+    /**
+        Returns false if this is a constant empty set of choices,
         which should not be modified.
     */
     bool IsOk() const
@@ -1471,78 +1442,47 @@ public:
         return ( m_data != wxPGChoicesEmptyData );
     }
 
+    /**
+        Returns item at given index.
+    */
     const wxPGChoiceEntry& Item( unsigned int i ) const
     {
         wxASSERT( IsOk() );
         return *m_data->Item(i);
     }
 
+    /**
+        Returns item at given index.
+    */
     wxPGChoiceEntry& Item( unsigned int i )
     {
         wxASSERT( IsOk() );
         return *m_data->Item(i);
     }
 
-    /** Removes count items starting at position nIndex. */
+    /**
+        Removes count items starting at position nIndex.
+    */
     void RemoveAt(size_t nIndex, size_t count = 1);
 
-    /** Does not create copies for itself. */
-    void Set( const wxChar** labels, const long* values = NULL )
-    {
-        Free();
-        Add(labels,values);
-    }
-
-    /** Version that works with wxArrayString.
-        TODO: Deprecate this.
+    /**
+        Sets contents from lists of strings and values.
     */
-    void Set( wxArrayString& arr, const long* values = (const long*) NULL )
-    {
-        Free();
-        Add(arr,values);
-    }
-
-    /** Version that works with wxArrayString and wxArrayInt. */
-    void Set( const wxArrayString& labels, const wxArrayInt& values = wxArrayInt() )
-    {
-        Free();
-        if ( &values )
-            Add(labels,values);
-        else
-            Add(labels);
-    }
-
-    // Creates exclusive copy of current choices
-    void SetExclusive()
-    {
-        if ( m_data->m_refCount != 1 )
-        {
-            wxPGChoicesData* data = new wxPGChoicesData();
-            data->CopyDataFrom(m_data);
-            Free();
-            m_data = data;
-        }
-    }
-
-    // Returns data, increases refcount.
-    wxPGChoicesData* GetData()
-    {
-        wxASSERT( m_data->m_refCount != 0xFFFFFFF );
-        m_data->m_refCount++;
-        return m_data;
-    }
+    void Set( const wxChar** labels, const long* values = NULL );
 
-    // Returns plain data ptr - no refcounting stuff is done.
-    wxPGChoicesData* GetDataPtr() const { return m_data; }
+    /**
+        Sets contents from lists of strings and values.
+    */
+    void Set( const wxArrayString& labels, const wxArrayInt& values = wxArrayInt() );
 
-    // Changes ownership of data to you.
-    wxPGChoicesData* ExtractData()
-    {
-        wxPGChoicesData* data = m_data;
-        m_data = wxPGChoicesEmptyData;
-        return data;
-    }
+    /**
+        Creates exclusive copy of current choices.
+    */
+    void SetExclusive();
 
+    /**
+        Returns array of choice labels.
+    */
     wxArrayString GetLabels() const;
 
     void operator= (const wxPGChoices& a)