[Marble-devel] Review Request: Use ViewportParams rather than AbstractProjection for projection calculations regarding the visible area

Bernhard Beschow bbeschow at cs.tu-berlin.de
Wed Oct 6 17:29:28 CEST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5539/
-----------------------------------------------------------

Review request for marble.


Summary
-------

Using ViewportParams rather than AbstractProjection simpliefies statements like viewport->currentProjection()->screenCoordinates(...,viewport,...) to viewport->screenCoordinates(...).

That way, in the future, ViewportParams could be used for projection calculations that need to take the current viewport (and possibly an elevation model) into account, while AbstractProjections represent the pure mathematical concept (i.e. no elevation model).


Diffs
-----

  /trunk/KDE/kdeedu/marble/src/lib/GeoPainter.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/MarbleMap.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/MarblePhysics.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/MarbleWidget.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/MarbleWidgetInputHandler.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/MeasureTool.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/PlacemarkLayout.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/PlacemarkPainter.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/Projections/MercatorProjection.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/VectorMap.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/ViewParams.h 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/ViewParams.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/ViewportParams.h 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/ViewportParams.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/graphicsview/MarbleGraphicsItem.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/graphicsview/MarbleGraphicsItem_p.h 1183221 
  /trunk/KDE/kdeedu/marble/src/lib/graphicsview/ScreenGraphicsItem_p.h 1183221 
  /trunk/KDE/kdeedu/marble/src/plasmoid/worldclock.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/plugins/render/crosshairs/CrosshairsPlugin.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/plugins/render/mapscale/MapScaleFloatItem.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/plugins/render/overviewmap/OverviewMap.cpp 1183221 
  /trunk/KDE/kdeedu/marble/src/plugins/render/positionmarker/PositionMarker.cpp 1183221 

Diff: http://svn.reviewboard.kde.org/r/5539/diff


Testing
-------

Works for me (KDE version of Marble).


Thanks,

Bernhard

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/marble-devel/attachments/20101006/b43677a3/attachment.htm 


More information about the Marble-devel mailing list