Can i still merge this, small KDirWatch improvement.
Albert Astals Cid
aacid at kde.org
Wed Aug 7 17:37:40 UTC 2013
El Dimecres, 7 d'agost de 2013, a les 13:13:36, Mark va escriure:
> On Wed, Aug 7, 2013 at 10:38 AM, Albert Astals Cid <aacid at kde.org> wrote:
> >> Hi Release team,
> >
> > Hi
> >
> >> I just got the OK to ship [1] but considering the very close release
> >> of KDE 4.11 it seems right to ask the release team for permission as
> >> well.
> >>
> >> The patch simply prevents one stat call with KDirWatch and makes the
> >> code a little bit better to understand. Tests pass just fine and othe
> >> additional (visual and gdb) testing hasn't shown any regression.
> >>
> >> If i am not allowed to push it (due to the very close 4.11 tag), when
> >> can i push it then? Right after the tag i assume?
> >
> > Honestly, if this is just a [small] speed optimization and doesn't fix any
> > bug, i don't see it belonging to 4.11 at all, just commit it to master
> > and leave it for 4.12.
> >
> > Unless is a so huge optimization our users are going to send us money for
> > the speed difference :D
> >
> > Cheers,
> >
> > Albert
>
> Hi Albert,
>
> I'm OK with this not going into 4.11.0 since that is just a too short
> notice. But i do like to get this in the next bugfix release (4.11.1). It
> might not be a strict "bugfix", but that's only because nobody ever
> opened a bug report for it.
> Specially because 4.11 is LTS! Users and likely some company's are
> going to be running that version for years to come. They should have
> this fix.
Let me repease, 4.11 is LTS *only for kde-workspace*. This patch is for
kdelibs, which will have a regular 4.12 release, so they'll get this
optimization in 5 months.
Cheers,
Albert
>
> No, it won't get you or KDE any money, but it does save one stat call
> in a very common code path (creating files) thus you could argue that
> this patch will extend the life of storage devices ;-)
>
> >> Kind regards,
> >> Mark
> >>
> >> [1] https://git.reviewboard.kde.org/r/111870/
>
> _______________________________________________
> release-team mailing list
> release-team at kde.org
> https://mail.kde.org/mailman/listinfo/release-team
More information about the release-team
mailing list