----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101081/#review2554 -----------------------------------------------------------
are you sure the property binding still works? ime, even if the qt doc about it isn't clear at all, any signal set as NOTIFY of some qproperty does seem to work only if doesn't have any parameter - Marco On April 11, 2011, 10:56 a.m., Anton Kreuzkamp wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101081/ > ----------------------------------------------------------- > > (Updated April 11, 2011, 10:56 a.m.) > > > Review request for Plasma and Marco Martin. > > > Summary > ------- > > Add the name of the changed source as parameter to the dataChanged() signal > in the qml DataSource class. > This eases up the qml-code massively and improves the performance, as I don't > need to update all sources anymore, when just one source changes. > > > Diffs > ----- > > plasma/declarativeimports/core/datasource.cpp 0cd5a73 > plasma/declarativeimports/core/datasource_p.h 2d2cb13 > > Diff: http://git.reviewboard.kde.org/r/101081/diff > > > Testing > ------- > > Works. > > > Thanks, > > Anton > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
