On 04/06/2014 11:12 PM, Tobias Doerffel wrote:
> 2014-04-05 13:44 GMT+02:00 Vesa <[email protected]>:
>>>     QColor lineColor() const;
>>>     void setlineColor( const QColor & _c );
>>>     ...
>>>     QColor * m_lineColor; //!< unused yet
>>>     ...
>>>     Q_PROPERTY(QColor lineColor READ lineColor WRITE setlineColor)
>>>
>>> So this is the same as for outerColor. Is this what you need?
>> Yeah. That works.
> No, m_lineColor shouldn't be a pointer but a regular class instance
> instead. (Besides this: coding style! -> setLineColor( const QColor& c
> );
>
> The rest looks good, go ahead! :-)

Why not a pointer? All the other QColor qproperties are stored as
pointers to QColor...

That's how it was done for the outerColor property of the knob class, so
I also used the same way for the track properties...

Should these be changed?

------------------------------------------------------------------------------
_______________________________________________
LMMS-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/lmms-devel

Reply via email to