Review Request: Normalizing SIGNAL and SLOT signatures

Torsten Rahn tackat at kde.org
Thu Jan 3 18:46:54 UTC 2013



> On Jan. 3, 2013, 6:38 p.m., Torsten Rahn wrote:
> > Ship It!
> 
> Torsten Rahn wrote:
>     Looks good - except for the spaces. But I guess for this time we can ignore them.

Ooops, creates an error still:

[ 18%] Building CXX object src/lib/CMakeFiles/marblewidget.dir/geodata/data/GeoDataPoint.o
[ 18%] Building CXX object src/lib/CMakeFiles/marblewidget.dir/geodata/data/GeoDataLineString.o
[ 18%] Building CXX object src/lib/CMakeFiles/marblewidget.dir/geodata/data/GeoDataLinearRing.o
/home/mayank/marble/src/marble/src/lib/geodata/data/GeoDataLinearRing.cpp: In member function ‘virtual QVector<Marble::GeoDataLineString*> Marble::GeoDataLinearRing::toRangeCorrected() const’:
/home/mayank/marble/src/marble/src/lib/geodata/data/GeoDataLinearRing.cpp:58:67: error: conversion from ‘Marble::GeoDataLineString’ to non-scalar type ‘Marble::GeoDataLinearRing’ requested
/home/mayank/marble/src/marble/src/lib/geodata/data/GeoDataLinearRing.cpp:59:66: error: no match for 


- Torsten


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108094/#review24597
-----------------------------------------------------------


On Jan. 3, 2013, 6:24 p.m., Mayank Madan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108094/
> -----------------------------------------------------------
> 
> (Updated Jan. 3, 2013, 6:24 p.m.)
> 
> 
> Review request for KDE Edu, Kevin Krammer and Torsten Rahn.
> 
> 
> Description
> -------
> 
> Fixing SIGNAL/SLOT signatures to normalized form as reported by Krazy
> 
> 
> Diffs
> -----
> 
>   src/lib/MarbleMap.cpp 14f20dc 
>   src/lib/TileCoordsPyramid.h b669e6c 
>   src/lib/geodata/data/GeoDataContainer.h 8cb94cb 
>   src/lib/geodata/data/GeoDataLatLonAltBox.h 7276842 
>   src/lib/geodata/data/GeoDataLineString.h 1f833d0 
>   src/lib/geodata/data/GeoDataLinearRing.h 9cfce6d 
>   src/lib/geodata/data/GeoDataMultiGeometry.h 00f775f 
>   src/lib/geodata/data/GeoDataMultiTrack.h d929da8 
>   src/lib/geodata/data/GeoDataPlacemark.h 5dcfe6a 
>   src/lib/geodata/data/GeoDataPoint.h 2e8df39 
>   src/lib/geodata/data/GeoDataPolygon.h 788f19e 
>   src/lib/geodata/data/GeoDataTrack.h 98d8819 
>   src/plugins/declarative/Bookmarks.cpp 4319aab 
>   src/plugins/declarative/DeclarativeDataPlugin.cpp 8cd54f9 
>   src/plugins/declarative/MapThemeModel.cpp 47bc226 
>   src/plugins/declarative/Navigation.cpp 8636101 
>   src/plugins/declarative/OfflineDataModel.cpp 181319a 
>   src/plugins/declarative/PositionSource.cpp 03bb401 
>   src/plugins/declarative/RouteRequestModel.cpp aae7ff1 
>   src/plugins/declarative/Routing.cpp 7147d47 
>   src/plugins/declarative/Search.cpp 5bf98e2 
>   src/plugins/render/navigation/ArrowDiscWidget.h ad88852 
>   src/plugins/render/satellites/SatellitesMSCItem.cpp 465d1b9 
>   src/plugins/render/satellites/TrackerPluginItem.h c88e1ef 
>   src/plugins/render/twitter/twitterPlugin.cpp 81cbc30 
>   src/plugins/render/weather/BBCParser.h e1ca1e5 
>   src/plugins/runner/json/JsonParser.h 947f7df 
>   src/plugins/runner/json/JsonParser.cpp fc5eadf 
>   src/plugins/runner/local-osm-search/DatabaseQuery.h 0f8809d 
>   src/plugins/runner/local-osm-search/OsmDatabase.cpp 12d027e 
>   src/plugins/runner/osm/handlers/OsmNodeFactory.h 25d17ad 
>   src/plugins/runner/osm/handlers/OsmNodeFactory.cpp df25c95 
>   src/plugins/runner/osm/handlers/OsmRelationFactory.h a4c5f61 
>   src/plugins/runner/osm/handlers/OsmRelationFactory.cpp f1e9fdc 
>   src/plugins/runner/osm/handlers/OsmRelationTagHandler.cpp 892df17 
>   src/plugins/runner/osm/handlers/OsmWayFactory.h d3f3126 
>   src/plugins/runner/osm/handlers/OsmWayFactory.cpp a709ae5 
>   src/plugins/runner/osm/handlers/OsmWayTagHandler.cpp 1fc753c 
>   src/plugins/runner/pn2/Pn2Runner.cpp 767911f 
> 
> Diff: http://git.reviewboard.kde.org/r/108094/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mayank Madan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20130103/09378086/attachment.html>


More information about the kde-edu mailing list