From 749a0cdf3ecb0d736decd61870e831ca877081a5 Mon Sep 17 00:00:00 2001 From: dev Date: Fri, 8 Jun 2018 20:51:44 +0800 Subject: [PATCH] C++11 will automatically judge right-side reference when current function is returning. std::move is removed from return keyword. --- qtviewer_planetosm/osmtiles/layer_interface.h | 2 +- qtviewer_planetosm/qtaxviewer_planetosm.cpp | 4 +- qtviewer_planetosm/qtwidget_planetosm.cpp | 4 +- qtvplugin_geomarker/geographicsscene.cpp | 4 +- qtvplugin_geomarker/geoitembase.cpp | 4 +- .../qtvplugin_geomarker_func.cpp | 102 +++++++++--------- .../qtvplugin_geomarker_xml.cpp | 2 +- qtvplugin_grid/qtvplugin_grid.cpp | 2 +- test_container/testcontainer.cpp | 4 +- test_container/testcontainer_linux.cpp | 4 +- 10 files changed, 66 insertions(+), 66 deletions(-) diff --git a/qtviewer_planetosm/osmtiles/layer_interface.h b/qtviewer_planetosm/osmtiles/layer_interface.h index 4e33e99..66963ac 100644 --- a/qtviewer_planetosm/osmtiles/layer_interface.h +++ b/qtviewer_planetosm/osmtiles/layer_interface.h @@ -94,7 +94,7 @@ namespace QTVOSM virtual void set_visible(bool /*vb*/) {} public: //user-def direct function calls - virtual QMap call_func(const QMap & /*paras*/){return std::move( QMap());} + virtual QMap call_func(const QMap & /*paras*/){return ( QMap());} }; diff --git a/qtviewer_planetosm/qtaxviewer_planetosm.cpp b/qtviewer_planetosm/qtaxviewer_planetosm.cpp index 277fee4..804d002 100644 --- a/qtviewer_planetosm/qtaxviewer_planetosm.cpp +++ b/qtviewer_planetosm/qtaxviewer_planetosm.cpp @@ -416,7 +416,7 @@ QString qtaxviewer_planetosm::map_to_string(const QMap & m) s += p.value().toString(); s += ";"; } - return std::move(s); + return /*std::move*/(s); } QMap qtaxviewer_planetosm::string_to_map(const QString & s) @@ -433,7 +433,7 @@ QMap qtaxviewer_planetosm::string_to_map(const QString & s) res[name] = value; } } - return std::move(res); + return /*std::move*/(res); } /** diff --git a/qtviewer_planetosm/qtwidget_planetosm.cpp b/qtviewer_planetosm/qtwidget_planetosm.cpp index e43e1e6..cc6542a 100644 --- a/qtviewer_planetosm/qtwidget_planetosm.cpp +++ b/qtviewer_planetosm/qtwidget_planetosm.cpp @@ -391,7 +391,7 @@ QString qtwidget_planetosm::map_to_string(const QMap & m) s += p.value().toString(); s += ";"; } - return std::move(s); + return /*std::move*/(s); } QMap qtwidget_planetosm::string_to_map(const QString & s) @@ -408,7 +408,7 @@ QMap qtwidget_planetosm::string_to_map(const QString & s) res[name] = value; } } - return std::move(res); + return /*std::move*/(res); } /** * @brief osm_layer_call_function call layers' call_func method from diff --git a/qtvplugin_geomarker/geographicsscene.cpp b/qtvplugin_geomarker/geographicsscene.cpp index 79cfaa7..bbbdcda 100644 --- a/qtvplugin_geomarker/geographicsscene.cpp +++ b/qtvplugin_geomarker/geographicsscene.cpp @@ -123,10 +123,10 @@ namespace QTVP_GEOMARKER{ QList geoGraphicsScene::geo_items() { - return std::move(m_map_items.values()); + return /*std::move*/(m_map_items.values()); } QList geoGraphicsScene::geo_item_names() { - return std::move(m_map_items.keys()); + return /*std::move*/(m_map_items.keys()); } } diff --git a/qtvplugin_geomarker/geoitembase.cpp b/qtvplugin_geomarker/geoitembase.cpp index 59cab5d..9dd11cc 100644 --- a/qtvplugin_geomarker/geoitembase.cpp +++ b/qtvplugin_geomarker/geoitembase.cpp @@ -96,7 +96,7 @@ namespace QTVP_GEOMARKER{ QStringList geoItemBase::prop_names() { - return std::move(m_props.keys()); + return /*std::move*/(m_props.keys()); } int geoItemBase::prop_counts() { @@ -104,7 +104,7 @@ namespace QTVP_GEOMARKER{ } QVariantList geoItemBase::prop_values() { - return std::move(m_props.values()); + return /*std::move*/(m_props.values()); } const QVariant & geoItemBase::prop_data(const QString & k) diff --git a/qtvplugin_geomarker/qtvplugin_geomarker_func.cpp b/qtvplugin_geomarker/qtvplugin_geomarker_func.cpp index e467409..4699e1f 100644 --- a/qtvplugin_geomarker/qtvplugin_geomarker_func.cpp +++ b/qtvplugin_geomarker/qtvplugin_geomarker_func.cpp @@ -53,7 +53,7 @@ QMap qtvplugin_geomarker::call_func(const QMap qtvplugin_geomarker::func_update_point (const QMapgeoitem_by_name(name); QPen pen (m_default_style.pen); @@ -295,7 +295,7 @@ QMap qtvplugin_geomarker::func_update_point (const QMap qtvplugin_geomarker:: func_update_icon (const QMapgeoitem_by_name(name); QString icon_name = m_default_style.icon_name; @@ -420,7 +420,7 @@ QMap qtvplugin_geomarker:: func_update_icon (const QMap qtvplugin_geomarker::func_update_line (const QMapgeoitem_by_name(name); QPen pen (m_default_style.pen); @@ -555,7 +555,7 @@ QMap qtvplugin_geomarker::func_update_line (const QMap qtvplugin_geomarker::func_update_polygon (const QMap qtvplugin_geomarker::func_update_polygon (const QMap qtvplugin_geomarker::func_update_polygon (const QMap qtvplugin_geomarker::func_update_polygon (const QMap qtvplugin_geomarker::func_update_props(const QMap qtvplugin_geomarker::func_update_props(const QMap qtvplugin_geomarker::func_exists (const QMapgeoitem_by_name(name); if (base) @@ -834,7 +834,7 @@ QMap qtvplugin_geomarker::func_exists (const QMap qtvplugin_geomarker::func_delete_marks (const QMap qtvplugin_geomarker::func_delete_props (const QMap qtvplugin_geomarker::func_delete_props (const QMap qtvplugin_geomarker::func_mark_names (const QMap qtvplugin_geomarker::func_mark (const QMapgeoitem_by_name(name); if (item) @@ -1139,7 +1139,7 @@ QMap qtvplugin_geomarker::func_mark (const QMap qtvplugin_geomarker::func_props (const QMapgeoitem_by_name(name); if (item) @@ -1179,7 +1179,7 @@ QMap qtvplugin_geomarker::func_props (const QMap qtvplugin_geomarker::func_save_xml (const QMapxml_save(name); @@ -1224,13 +1224,13 @@ QMap qtvplugin_geomarker::func_load_xml (const QMapxml_load(name); @@ -1257,19 +1257,19 @@ QMap qtvplugin_geomarker::func_add_resource (const QMap qtvplugin_geomarker::func_add_resource (const QMap qtvplugin_geomarker::func_save_resources (const QMapxml_icon_save(name); @@ -1332,13 +1332,13 @@ QMap qtvplugin_geomarker::func_load_resources (const QMapxml_icon_load(name); @@ -1397,7 +1397,7 @@ QMap qtvplugin_geomarker::func_props_vis (const QMap qtvplugin_geomarker::func_show_props (const QMap qtvplugin_geomarker::func_set_mod (const QMap qtvplugin_geomarker::func_set_mod (const QMap qtvplugin_geomarker::func_selection_clear(const QMap res; this->on_pushButton_QTV_sel_clear_clicked(); - return std::move(res); + return /*std::move*/(res); } /** * @brief func_set_mod is a internal function for plugin call_func "selection_delete" @@ -1529,7 +1529,7 @@ QMap qtvplugin_geomarker::func_selection_delete(const QMap< { QMap res; this->on_pushButton_QTV_sel_delselected_clicked(); - return std::move(res); + return /*std::move*/(res); } /** * @brief func_set_mod is a internal function for plugin call_func "selected_items" @@ -1551,7 +1551,7 @@ QMap qtvplugin_geomarker::func_selected_items (const QMap qtvplugin_geomarker::func_set_default_style(const QMap qtvplugin_geomarker::func_default_style(const QMaplineEdit_QTV_icon_rotate->text(); //!smooth is the transform mode that this icon will use. 0 mean not smooth, but faster. 1 mean smooth. res["smooth"] = ui->checkBox_QTV_icon_smooth->isChecked()?-1:0; - return std::move(res); + return /*std::move*/(res); } diff --git a/qtvplugin_geomarker/qtvplugin_geomarker_xml.cpp b/qtvplugin_geomarker/qtvplugin_geomarker_xml.cpp index c09a61a..ff3d0c4 100644 --- a/qtvplugin_geomarker/qtvplugin_geomarker_xml.cpp +++ b/qtvplugin_geomarker/qtvplugin_geomarker_xml.cpp @@ -212,7 +212,7 @@ QMap qtvplugin_geomarker::xml_attribs_map(const QXmlStreamAttri QMap attr; foreach(QXmlStreamAttribute a, ats) attr[a.name().toString()] = a.value().toString(); - return std::move(attr); + return /*std::move*/(attr); } bool qtvplugin_geomarker::xml_load(QString xmlfile) { diff --git a/qtvplugin_grid/qtvplugin_grid.cpp b/qtvplugin_grid/qtvplugin_grid.cpp index 3ad9da6..53e4e6e 100644 --- a/qtvplugin_grid/qtvplugin_grid.cpp +++ b/qtvplugin_grid/qtvplugin_grid.cpp @@ -734,7 +734,7 @@ QMap qtvplugin_grid::call_func(const QMap } else res["error"] = "\"function\" keyword not specified, nothing to do."; - return std::move(res); + return /*std::move*/(res); } void qtvplugin_grid::on_pushButton_QTV_add_mark_clicked() { diff --git a/test_container/testcontainer.cpp b/test_container/testcontainer.cpp index e4d6840..5ba74bd 100644 --- a/test_container/testcontainer.cpp +++ b/test_container/testcontainer.cpp @@ -247,7 +247,7 @@ QString testcontainer::map_to_string(const QMap & m) s += p.value().toString(); s += ";"; } - return std::move(s); + return /*std::move*/(s); } QMap testcontainer::string_to_map(const QString & s) @@ -264,7 +264,7 @@ QMap testcontainer::string_to_map(const QString & s) res[name] = value; } } - return std::move(res); + return /*std::move*/(res); } void testcontainer::on_pushButton_QTV_test_grid_enable_clicked() { diff --git a/test_container/testcontainer_linux.cpp b/test_container/testcontainer_linux.cpp index 17ca666..91598b1 100644 --- a/test_container/testcontainer_linux.cpp +++ b/test_container/testcontainer_linux.cpp @@ -195,7 +195,7 @@ QString testcontainer::map_to_string(const QMap & m) s += p.value().toString(); s += ";"; } - return std::move(s); + return /*std::move*/(s); } QMap testcontainer::string_to_map(const QString & s) @@ -212,7 +212,7 @@ QMap testcontainer::string_to_map(const QString & s) res[name] = value; } } - return std::move(res); + return /*std::move*/(res); } void testcontainer::on_pushButton_QTV_test_grid_enable_clicked() { -- GitLab