[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    Re: kspread: non-contiguous selection
From:       Tomas Mecir <mecirt () gmail ! com>
Date:       2005-12-17 13:08:00
Message-ID: 492258b10512170508q5416dbceue9bcb33c23e64f22 () mail ! gmail ! com
[Download RAW message or body]

On 12/17/05, Stefan Nikolaus <stefan.nikolaus@kdemail.net> wrote:
> On Saturday, 17. December 2005 13:24, Tomas Mecir wrote:
> > Ah, so basically, you do it to make things run faster ?
> At least the add() method looks, if the region already contains the
> range/point. And I want to save memory. ;)

I see, okies :)

> > So shall it be then. Does the basic Manipulator already use
> > begin/endOperation ? I could also look at the code, but this is faster
> >
> > :P
> > :
>
> Yes.

Very well.

> > > The manipulators themselves are all located in manipulator.cc|h. The
> > > CellWorkers should be replaced by Manipulators step by step.
> >
> > Ah, I'd say, let's keep only the manipulators that we want to inherit
> > from in manipulator.h - other should go to something like
> > formatting_manipulators.* and somesuch. Not really a big issue, but it
> > makes navigating over code easier - IMHO more smaller files (and
> > correctly organized) are better than a couple of 200K+ ones.
>
> Okay, but reverse the naming in order for faster finding these ones, i.e.
> manipulator_formatting.h. IMHO we could do this later with the
> ManipulatorManger, which already exists in some extend. ;-)

Sure, sounds good :)
What does your "some extend" of a ManipulatorManager do ?

/ Tomas
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]