From f1e49038543cf75766f4a220f62cdc6cdbc0e27d Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Sat, 26 Jun 2021 21:35:18 +1000 Subject: Cleanup: full sentences in comments, improve comment formatting --- source/blender/io/collada/ArmatureImporter.cpp | 4 ++-- source/blender/io/collada/BCAnimationCurve.cpp | 2 +- source/blender/io/collada/BCAnimationSampler.cpp | 4 ++-- source/blender/io/collada/BCAnimationSampler.h | 2 +- source/blender/io/collada/DocumentImporter.h | 8 +++++--- source/blender/io/collada/GeometryExporter.cpp | 2 +- source/blender/io/collada/collada_utils.cpp | 6 +++--- source/blender/io/gpencil/intern/gpencil_io_export_svg.cc | 2 +- source/blender/io/usd/intern/usd_writer_mesh.cc | 2 +- 9 files changed, 17 insertions(+), 15 deletions(-) (limited to 'source/blender/io') diff --git a/source/blender/io/collada/ArmatureImporter.cpp b/source/blender/io/collada/ArmatureImporter.cpp index 5c3d6be589a..f36b9aacd8b 100644 --- a/source/blender/io/collada/ArmatureImporter.cpp +++ b/source/blender/io/collada/ArmatureImporter.cpp @@ -325,7 +325,7 @@ void ArmatureImporter::connect_bone_chains(bArmature *armature, BoneExtended *pbe = extended_bones[parentbone->name]; if (dominant_child != nullptr) { - /* Found a valid chain. Now connect current bone with that chain.*/ + /* Found a valid chain. Now connect current bone with that chain. */ EditBone *pebone = bc_get_edit_bone(armature, parentbone->name); EditBone *cebone = bc_get_edit_bone(armature, dominant_child->get_name()); if (pebone && !(cebone->flag & BONE_CONNECTED)) { @@ -341,7 +341,7 @@ void ArmatureImporter::connect_bone_chains(bArmature *armature, if (len_squared_v3(vec) > MINIMUM_BONE_LENGTH) { copy_v3_v3(pebone->tail, cebone->head); - pbe->set_tail(pebone->tail); /* to make fix_leafbone happy ...*/ + pbe->set_tail(pebone->tail); /* To make fix_leafbone happy. */ if (pbe && pbe->get_chain_length() >= this->import_settings->min_chain_length) { BoneExtended *cbe = extended_bones[cebone->name]; diff --git a/source/blender/io/collada/BCAnimationCurve.cpp b/source/blender/io/collada/BCAnimationCurve.cpp index 5065accf554..0e31e522ec4 100644 --- a/source/blender/io/collada/BCAnimationCurve.cpp +++ b/source/blender/io/collada/BCAnimationCurve.cpp @@ -314,7 +314,7 @@ void BCAnimationCurve::clean_handles() fcurve = get_edit_fcurve(); } - /* Keep old bezt data for copy)*/ + /* Keep old bezt data for copy). */ BezTriple *old_bezts = fcurve->bezt; int totvert = fcurve->totvert; fcurve->bezt = nullptr; diff --git a/source/blender/io/collada/BCAnimationSampler.cpp b/source/blender/io/collada/BCAnimationSampler.cpp index b2a9027380e..02a4e1f3167 100644 --- a/source/blender/io/collada/BCAnimationSampler.cpp +++ b/source/blender/io/collada/BCAnimationSampler.cpp @@ -488,7 +488,7 @@ void BCAnimationSampler::initialize_curves(BCAnimationCurveMap &curves, Object * } } - /* Add curves on Object->material actions*/ + /* Add curves on Object->material actions. */ object_type = BC_ANIMATION_TYPE_MATERIAL; for (int a = 0; a < ob->totcol; a++) { /* Export Material parameter animations. */ @@ -574,7 +574,7 @@ BCSample &BCSampleFrameContainer::add(Object *ob, int frame_index) /* Below are the getters which we need to export the data */ /* ====================================================== */ -/* Return either the BCSampleFrame or NULL if frame does not exist*/ +/* Return either the BCSampleFrame or NULL if frame does not exist. */ BCSampleFrame *BCSampleFrameContainer::get_frame(int frame_index) { BCSampleFrameMap::iterator it = sample_frames.find(frame_index); diff --git a/source/blender/io/collada/BCAnimationSampler.h b/source/blender/io/collada/BCAnimationSampler.h index 52c0c8aa29e..19b1bc35264 100644 --- a/source/blender/io/collada/BCAnimationSampler.h +++ b/source/blender/io/collada/BCAnimationSampler.h @@ -88,7 +88,7 @@ class BCSampleFrame { BCSample &add(Object *ob); - /* Following methods return NULL if object is not in the sampleMap*/ + /* Following methods return NULL if object is not in the sampleMap. */ const BCSample *get_sample(Object *ob) const; const BCMatrix *get_sample_matrix(Object *ob) const; const BCMatrix *get_sample_matrix(Object *ob, Bone *bone) const; diff --git a/source/blender/io/collada/DocumentImporter.h b/source/blender/io/collada/DocumentImporter.h index a6ed014a33c..8e553f1ce7a 100644 --- a/source/blender/io/collada/DocumentImporter.h +++ b/source/blender/io/collada/DocumentImporter.h @@ -77,11 +77,13 @@ class DocumentImporter : COLLADAFW::IWriter { */ void cancel(const COLLADAFW::String &errorMessage); - /** This is the method called. The writer hast to prepare to receive data.*/ + /** This is the method called. The writer hast to prepare to receive data. */ void start(); - /** This method is called after the last write* method. No other methods will be called after - * this.*/ + /** + * This method is called after the last write* method. + * No other methods will be called after this. + */ void finish(); bool writeGlobalAsset(const COLLADAFW::FileInfo *); diff --git a/source/blender/io/collada/GeometryExporter.cpp b/source/blender/io/collada/GeometryExporter.cpp index def7caabf32..382bb8a6e36 100644 --- a/source/blender/io/collada/GeometryExporter.cpp +++ b/source/blender/io/collada/GeometryExporter.cpp @@ -589,7 +589,7 @@ void GeometryExporter::createTexcoordsSource(std::string geom_id, Mesh *me) bool operator<(const Normal &a, const Normal &b) { - /* only needed to sort normal vectors and find() them later in a map.*/ + /* Only needed to sort normal vectors and find() them later in a map. */ return a.x < b.x || (a.x == b.x && (a.y < b.y || (a.y == b.y && a.z < b.z))); } diff --git a/source/blender/io/collada/collada_utils.cpp b/source/blender/io/collada/collada_utils.cpp index bba50064879..d7855d69d99 100644 --- a/source/blender/io/collada/collada_utils.cpp +++ b/source/blender/io/collada/collada_utils.cpp @@ -636,7 +636,7 @@ void BoneExtended::set_bone_layers(std::string layerString, std::vector> layer) { - /* Blender uses numbers to specify layers*/ + /* Blender uses numbers to specify layers. */ if (isInteger(layer)) { pos = atoi(layer.c_str()); if (pos >= 0 && pos < 32) { @@ -645,10 +645,10 @@ void BoneExtended::set_bone_layers(std::string layerString, std::vector= layer_labels.size()) { - layer_labels.push_back(layer); /* remember layer number for future usage*/ + layer_labels.push_back(layer); /* Remember layer number for future usage. */ } if (pos > 31) { diff --git a/source/blender/io/gpencil/intern/gpencil_io_export_svg.cc b/source/blender/io/gpencil/intern/gpencil_io_export_svg.cc index 438263167ca..a9745415d40 100644 --- a/source/blender/io/gpencil/intern/gpencil_io_export_svg.cc +++ b/source/blender/io/gpencil/intern/gpencil_io_export_svg.cc @@ -295,7 +295,7 @@ void GpencilExporterSVG::export_stroke_to_path(bGPDlayer *gpl, const float2 screen_co = gpencil_3D_point_to_2D(&pt.x); txt.append(std::to_string(screen_co.x) + "," + std::to_string(screen_co.y)); } - /* Close patch (cyclic)*/ + /* Close patch (cyclic). */ if (gps->flag & GP_STROKE_CYCLIC) { txt.append("z"); } diff --git a/source/blender/io/usd/intern/usd_writer_mesh.cc b/source/blender/io/usd/intern/usd_writer_mesh.cc index 17c41e0edc1..54316e56867 100644 --- a/source/blender/io/usd/intern/usd_writer_mesh.cc +++ b/source/blender/io/usd/intern/usd_writer_mesh.cc @@ -165,7 +165,7 @@ void USDGenericMeshWriter::write_mesh(HierarchyContext &context, Mesh *mesh) /* The material path will be of the form , which is outside the * sub-tree pointed to by ref_path. As a result, the referenced data is not allowed to point * out of its own sub-tree. It does work when we override the material with exactly the same - * path, though.*/ + * path, though. */ if (usd_export_context_.export_params.export_materials) { assign_materials(context, usd_mesh, usd_mesh_data.face_groups); } -- cgit v1.2.3