Re: grantlee-0.1.8 build failed on arm7

2011-06-21 Thread Rolf Eike Beer
Sune Vuorela wrote: diff --git a/templates/lib/abstractlocalizer.cpp b/templates/lib/abstractlocalizer.cpp index 4e5b15d..104d888 100644 --- a/templates/lib/abstractlocalizer.cpp +++ b/templates/lib/abstractlocalizer.cpp @@ -46,8 +46,8 @@ QString AbstractLocalizer::localize( const QVariant

Re: grantlee-0.1.8 build failed on arm7

2011-06-21 Thread Sune Vuorela
On 2011-06-21, Rolf Eike Beer k...@opensource.sf-tec.de wrote: So if it is a double you are truncating it to a float (on ARM). I don't know if that is intentional. Given the api is taking qreal's, I think it is intentional. /Sune

Re: grantlee-0.1.8 build failed on arm7

2011-06-21 Thread Harald Sitter
On Tue, Jun 21, 2011 at 9:20 AM, Rolf Eike Beer k...@opensource.sf-tec.de wrote: Sune Vuorela wrote: diff --git a/templates/lib/abstractlocalizer.cpp b/templates/lib/abstractlocalizer.cpp index 4e5b15d..104d888 100644 --- a/templates/lib/abstractlocalizer.cpp +++

Re: Review Request: konqueror: reset URL when pressing ESC in address bar

2011-06-21 Thread Marcel Partap
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6681/ --- (Updated June 21, 2011, 11:26 a.m.) Review request for kdelibs and David

Re: Review Request: konqueror: reset URL when pressing ESC in address bar

2011-06-21 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6681/#review10245 --- Looks good, apart from some details.

Re: Review Request: Draw overlays even for previews.

2011-06-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101570/#review4059 --- This review has been submitted with commit

Re: Review Request: Adds KIconLoader::drawOverlays which allows to draw overlays on any pixmap.

2011-06-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101569/#review4060 --- This review has been submitted with commit

Re: Review Request: Show icon overlays in the Informationen Panel.

2011-06-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101573/#review4061 --- This review has been submitted with commit

Re: Review Request: konqueror: reset URL when pressing ESC in address bar

2011-06-21 Thread Dawit A
There is one particular issue with this patch. When you first start konqueror, the introduction page is shown with a blank location bar. If you then simply press escape in location bar, the location bar switches to about:konqueror because that is the url of the current view. Perhaps the patch

Re: Review Request: Add missing dependency to xmllint

2011-06-21 Thread Sune Vuorela
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101707/#review4053 --- Ship it! Looks right to me. Thanks for fixing. - Sune On

Re: Review Request: konqueror: reset URL when pressing ESC in address bar

2011-06-21 Thread Marcel Partap
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6681/ --- (Updated June 21, 2011, 4:20 p.m.) Review request for kdelibs and David

Re: Review Request: konqueror: reset URL when pressing ESC in address bar

2011-06-21 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6681/#review10246 --- OK for url() then. Re the code above, you're probably right, it

Re: Review Request: Add missing dependency to xmllint

2011-06-21 Thread Luigi Toscano
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101707/ --- (Updated June 21, 2011, 7:21 p.m.) Review request for Build System,

Re: Review Request: Add missing dependency to xmllint

2011-06-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101707/#review4064 --- This review has been submitted with commit