> On Feb. 22, 2013, 12:41 a.m., Àlex Fiestas wrote:
> > Tested the patch with chromiium 24.0.1312.70 (181759) worked fine.
> >
> > Code wise it looks fine as well.
Thanks!
Applied to branches KDE/4.10, and master, I guess it's enough.
- Marco
---
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109049/#review27881
---
This review has been submitted with commit
9fb3c1daa5879f30c29
> On Feb. 18, 2013, 1:39 a.m., Aleix Pol Gonzalez wrote:
> > I don't see loosening the variables' scope as a codebase improvement.
> > Mostly otherwise.
> >
> > Also I'd like to know how you measured this 5% of improvement, which either
> > way I'm unsure if it's worth it considering that this
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109049/#review27876
---
Ship it!
Tested the patch with chromiium 24.0.1312.70 (181759
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109049/
---
(Updated Feb. 21, 2013, 7:29 p.m.)
Review request for kde-workspace and Pl
> On Feb. 20, 2013, 1:06 a.m., Àlex Fiestas wrote:
> > I guess this will break compatibility with old versions then? if so, do you
> > know from which version will it work?
> > If this was not long ago, can we check the version and keep supporting the
> > old and the new one?
>
> Àlex Fiestas
Hi,
> On February 18th, 2013, 1:39 a.m. UTC, Aleix Pol Gonzalez wrote:
>
> I don't see loosening the variables' scope as a codebase improvement. Mostly
> otherwise.
>
> Also I'd like to know how you measured this 5% of improvement, which either
> way I'm unsure if it's worth it considering that