Change Details

The goal of this patch is to encapsulate classes derived from MyMoneyObject. It follows KDE's Library Code Policy. It considers following classes: MyMoneyAccount MyMoneySecurity MyMoneyTag MyMoneySchedule MyMoneyPayee Changes: 1) d-pointers to move includes for non-PODs to source files, for now they are only for data members, 2) all accessors in source file and handled through d-pointer, to keep all data members in one place i.e. in PIMPL, 3) more forward-declarations. Changes pattern is the same for all classes. Little code changes e.g. accountList is now returned by value and not reference, so some loops had to be redesigned.

The goal of this patch is to encapsulate classes derived from MyMoneyObject. It follows KDE's Library Code Policy. It considers all classes derived from MyMoneyObject. Changes: 1) d-pointers to move includes for non-PODs to source files, for now they are only for data members, 2) all accessors in source file and handled through d-pointer, to keep all data members in one place i.e. in PIMPL, 3) more forward-declarations. Changes pattern is the same for all classes. Little code changes e.g. some loops use foreach instead of iterators. It avoids crashes.

The goal of this patch is to encapsulate classes derived from MyMoneyObject. It follows KDE's Library Code Policy. It considers followingall classes: MyMoneyAccount MyMoneySecurity MyMoneyTag derived from MyMoneySchedule MyMoneyPayeeeyObject. Changes: 1) d-pointers to move includes for non-PODs to source files, for now they are only for data members, 2) all accessors in source file and handled through d-pointer, to keep all data members in one place i.e. in PIMPL, 3) more forward-declarations. Changes pattern is the same for all classes. Little code changes e.g. accountList is now returned by value and not reference,some loops use foreach instead of iterators. so some loops had to be redesignedIt avoids crashes.