[Kst] Re: branches/work/kst/portto4/kst

[Barth Netterfield]

Peter,

Can you outline your plan in a little more detail.
Are you planning to replace KstSharedPtr?  If so, this needs to be discussed.

cbn

On Sat, Jan 1, 2011 at 6:51 AM, Peter Kümmel <syntheticpp at gmx.net> wrote:
> SVN commit 1210492 by kuemmel:
>
> Start port to QSharedPointer:
> - the current live time handling is no standard C++ idiom
> - implementing a shared pointer is legacy code from KDE 3
> - QSharedPointer is a bullet proofed Qt class
> - better encapsulation: objects should not know how they are stored (remove inheritance from Shared)
> - define KST_USE_QSHAREDPOINTER in sharedptr.h to enable new code
>
> One problem is the pointer -> SharedPointer transition, because mostly there exists already
> a QSharedPointer for the object Qt asserts. Could be solved with a consequent usage of shared pointers.
> Don't know why raw pointers are used.
>
>
>  M  +12 -0     devel-docs/Kst2Specs/Wishlist
>  M  +2 -0      src/libkst/datamatrix.cpp
>  M  +5 -0      src/libkst/datamatrix.h
>  M  +4 -0      src/libkst/dataplugin.h
>  M  +4 -0      src/libkst/datavector.cpp
>  M  +6 -0      src/libkst/datavector.h
>  M  +6 -0      src/libkst/matrix.h
>  M  +9 -1      src/libkst/object.cpp
>  M  +9 -1      src/libkst/object.h
>  M  +11 -0     src/libkst/objectstore.h
>  M  +13 -0     src/libkst/primitive.h
>  M  +37 -1     src/libkst/sharedptr.h
>  M  +5 -2      src/libkst/timezones.h
>  M  +6 -0      src/libkst/vector.h
>  M  +5 -0      src/libkstmath/basicplugin.h
>  M  +6 -0      src/libkstmath/csd.h
>  M  +5 -0      src/libkstmath/curve.h
>  M  +1 -1      src/libkstmath/curvehint.cpp
>  M  +4 -0      src/libkstmath/curvehint.h
>  M  +8 -8      src/libkstmath/dataobject.cpp
>  M  +6 -0      src/libkstmath/dataobject.h
>  M  +6 -0      src/libkstmath/equation.h
>  M  +6 -0      src/libkstmath/eventmonitorentry.h
>  M  +6 -0      src/libkstmath/histogram.h
>  M  +6 -0      src/libkstmath/image.h
>  M  +6 -0      src/libkstmath/psd.h
>  M  +8 -8      src/libkstmath/relation.cpp
>  MM +6 -0      src/plugins/filters/cumulativesum/cumulativesum.h
>  M  +4 -0      src/widgets/matrixselector.cpp
>  M  +4 -0      src/widgets/scalarselector.cpp
>  M  +4 -0      src/widgets/stringselector.cpp
>  M  +4 -0      src/widgets/vectorselector.cpp
>
>
> http://websvn.kde.org/?view=rev&revision=1210492
> _______________________________________________
> Kst mailing list
> Kst at kde.org
> https://mail.kde.org/mailman/listinfo/kst
>



-- 
C. Barth Netterfield
University of Toronto
416-845-0946