[Marble-devel] Review Request 114848: Fix QString doublequote_chars issues

Benjamin Kaiser benjaminjkaiser at gmail.com
Sat Jan 4 11:04:07 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114848/
-----------------------------------------------------------

Review request for Marble, Dennis Nienhüser and Torsten Rahn.


Repository: marble


Description
-------

Fix QString doublequote_chars issues as found by krazy.

The file /src/lib/marble/geodata/handlers/kml/KmlElementDictionary.cpp has been left unchanged. Dennis confirmed this was okay.

Krazy report: http://ebn.kde.org/krazy/reports/kde-4.x/kdeedu/marble/index.html
GCI Task: http://www.google-melange.com/gci/task/view/google/gci2013/5228673925906432


Diffs
-----

  src/lib/marble/MapThemeDownloadDialog.cpp 6a527a5 
  src/lib/marble/MapViewWidget.cpp 3450f71 
  src/lib/marble/MarbleLegendBrowser.cpp 644dd0f 
  src/lib/marble/MarbleWidgetPopupMenu.cpp e527bdf 
  src/lib/marble/MovieCapture.cpp 83b5c3e 
  src/lib/marble/MovieCaptureDialog.cpp 6e80d89 
  src/lib/marble/TemplateDocument.cpp 18a28d1 
  src/lib/marble/cloudsync/BookmarkSyncManager.cpp 43f93ad 
  src/lib/marble/cloudsync/OwncloudSyncBackend.cpp 889522e 
  src/lib/marble/cloudsync/RouteItemDelegate.cpp fc29a6a 
  src/lib/marble/cloudsync/RouteSyncManager.cpp c873ff1 
  src/plugins/render/opencachingcom/OpenCachingComItem.cpp d332c53 
  src/plugins/render/photo/PhotoPlugin.cpp acc7191 
  src/plugins/render/stars/StarsPlugin.cpp 51a2b65 
  src/plugins/runner/cyclestreets/CycleStreetsRunner.cpp 320635c 
  src/plugins/runner/kml/KmzHandler.cpp c20754f 
  tools/constellations2kml/constellations2kml.cpp a32d1f2 

Diff: https://git.reviewboard.kde.org/r/114848/diff/


Testing
-------

$ krazy2all --check doublequote_chars
=>c++/doublequote_chars test in-progress......................................................................................................................................................................................................done

krazy2 Analysis

Checkers Run = 1
Files Processed = 3176
Total Issues = 5 ...as of January 04 2014 21:55:49 EST

== c++: For File Type c++ ==
1. Check single-char QString operations for efficiency [doublequote_chars]... 5 issues found
	./src/lib/marble/geodata/handlers/kml/KmlElementDictionary.cpp: line#80,198,202,203,204 (5)
	Adding single characters to a QString is faster if the characters
	are QChars and not QStrings.  For example: QString path = oldpath +
	"/" + base is better written as QString path = oldpath + '/' +
	base. Same holds for arguments to QString::startsWith(),
	QString::endsWith(), QString::remove(), QString::section(), and
	QString::split(). Use QString::remove() instead of
	QString::replace(foo,"")


Thanks,

Benjamin Kaiser

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20140104/e965ca55/attachment.html>


More information about the Marble-devel mailing list