Yes.  I can remove them tonight.  I felt they enhanced readability,
especially when the context isn't clear, e.g. when searching.  It certainly
isn't crucial.  It might be the kind of thing that, in future interfaces, I
might write setFooThingy instead of setThingy.  Anyway I shouldn't have
been trying to change such things in this change set in the first place.
On Dec 8, 2013 5:26 PM, "David Capello" <notifications@github.com> wrote:

> In src/app/settings/settings.h:
>
> > @@ -93,6 +108,14 @@
> >    public:
> >      virtual ~IPenSettings() { }
> >
> > +    virtual raster::PenType getPenType() = 0;
> > +    virtual int getPenSize() = 0;
> > +    virtual int getPenAngle() = 0;
> > +
> > +    virtual void setPenType(PenType type) = 0;
> > +    virtual void setPenSize(int size) = 0;
> > +    virtual void setPenAngle(int angle) = 0;
> > +
>
> I don't like the explosion of methods here. Maybe a note that we should
> remove getType/getSize/getAngle() version. Anyway I don't know why or if we
> really need to change the name of the member functions. Can we just leave
> get/setType/Size/Angle?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/aseprite/aseprite/pull/9/files#r8186502>
> .
>