[Bf-blender-cvs] [22184f3aeef] master: Cleanup: spelling in comments

Campbell Barton noreply at git.blender.org
Mon Apr 4 23:50:34 CEST 2022


Commit: 22184f3aeef6181dd755579753ea6c92b2117578
Author: Campbell Barton
Date:   Tue Apr 5 07:39:40 2022 +1000
Branches: master
https://developer.blender.org/rB22184f3aeef6181dd755579753ea6c92b2117578

Cleanup: spelling in comments

===================================================================

M	intern/cycles/kernel/types.h
M	source/blender/editors/include/UI_view2d.h
M	source/blender/editors/interface/interface.cc
M	source/blender/editors/interface/interface_region_menu_pie.cc
M	source/blender/io/wavefront_obj/importer/obj_import_file_reader.cc

===================================================================

diff --git a/intern/cycles/kernel/types.h b/intern/cycles/kernel/types.h
index 9d9daaa0dda..db499a1e1bc 100644
--- a/intern/cycles/kernel/types.h
+++ b/intern/cycles/kernel/types.h
@@ -666,7 +666,7 @@ typedef struct AttributeDescriptor {
 /* For manifold next event estimation, we need space to store and evaluate
  * 2 closures (with extra data) on the refractive interfaces, in addition
  * to keeping the full sd at the current shading point. We need 4 because a
- * refractive bsdf is instanced with a companion reflection bsdf, even though
+ * refractive BSDF is instanced with a companion reflection BSDF, even though
  * we only need the refractive one, and each of them requires 2 slots. */
 #ifndef __CAUSTICS_MAX_CLOSURE__
 #  define CAUSTICS_MAX_CLOSURE 4
diff --git a/source/blender/editors/include/UI_view2d.h b/source/blender/editors/include/UI_view2d.h
index 56b0bd04a71..47c554f84e0 100644
--- a/source/blender/editors/include/UI_view2d.h
+++ b/source/blender/editors/include/UI_view2d.h
@@ -447,7 +447,7 @@ typedef struct View2DEdgePanData {
   struct ARegion *region;
   /** View2d we're operating in. */
   struct View2D *v2d;
-    /* Limit maximum pannable area */
+  /** Limit maximum pannable area. */
   struct rctf limit;
 
   /** Panning should only start once being in the inside rect once (e.g. adding nodes can happen
diff --git a/source/blender/editors/interface/interface.cc b/source/blender/editors/interface/interface.cc
index 9bd0498fcfb..b34d61d5192 100644
--- a/source/blender/editors/interface/interface.cc
+++ b/source/blender/editors/interface/interface.cc
@@ -6516,7 +6516,7 @@ uiBut *uiDefSearchButO_ptr(uiBlock *block,
 
   if (properties) {
     PointerRNA *ptr = UI_but_operator_ptr_get(but);
-    /* Copy idproperties. */
+    /* Copy id-properties. */
     ptr->data = IDP_CopyProperty(properties);
   }
 
@@ -6755,7 +6755,7 @@ void UI_but_string_info_get(bContext *C, uiBut *but, ...)
         PointerRNA *opptr = UI_but_operator_ptr_get(but);
         wmOperatorType *ot = but->optype;
 
-        /* so the context is passed to itemf functions */
+        /* So the context is passed to `itemf` functions. */
         WM_operator_properties_sanitize(opptr, false);
 
         /* if the default property of the operator is enum and it is set,
diff --git a/source/blender/editors/interface/interface_region_menu_pie.cc b/source/blender/editors/interface/interface_region_menu_pie.cc
index d30797cc4d0..b11564f09c5 100644
--- a/source/blender/editors/interface/interface_region_menu_pie.cc
+++ b/source/blender/editors/interface/interface_region_menu_pie.cc
@@ -332,7 +332,7 @@ static void ui_pie_menu_level_invoke(bContext *C, void *argN, void *arg2)
   PointerRNA ptr;
 
   WM_operator_properties_create_ptr(&ptr, lvl->ot);
-  /* so the context is passed to itemf functions (some need it) */
+  /* So the context is passed to `itemf` functions (some need it). */
   WM_operator_properties_sanitize(&ptr, false);
   PropertyRNA *prop = RNA_struct_find_property(&ptr, lvl->propname);
 
diff --git a/source/blender/io/wavefront_obj/importer/obj_import_file_reader.cc b/source/blender/io/wavefront_obj/importer/obj_import_file_reader.cc
index c528d220d04..d6020895fbf 100644
--- a/source/blender/io/wavefront_obj/importer/obj_import_file_reader.cc
+++ b/source/blender/io/wavefront_obj/importer/obj_import_file_reader.cc
@@ -470,7 +470,7 @@ static StringRef skip_unsupported_options(StringRef line)
     return line;
   }
 
-  /* Remove upto start of the last option + size of the last option + space after it. */
+  /* Remove up to start of the last option + size of the last option + space after it. */
   line = line.drop_prefix(last_option_pos + last_option.size() + 1);
   for (int i = 0; i < map_options.number_of_args(last_option); i++) {
     const int64_t pos_space{line.find_first_of(' ')};
@@ -589,7 +589,7 @@ void MTLParser::parse_and_store(Map<string, std::unique_ptr<MTLMaterial>> &r_mtl
 
     /* Parse image textures. */
     else if (line_key.find("map_") != StringRef::not_found) {
-      /* TODO howardt: fix this */
+      /* TODO(@howardt): fix this. */
       eMTLSyntaxElement line_key_enum = mtl_line_key_str_to_enum(line_key);
       if (line_key_enum == eMTLSyntaxElement::string ||
           !current_mtlmaterial->texture_maps.contains_as(line_key_enum)) {
@@ -601,7 +601,7 @@ void MTLParser::parse_and_store(Map<string, std::unique_ptr<MTLMaterial>> &r_mtl
       Vector<StringRef> str_map_xx_split;
       split_by_char(rest_line, ' ', str_map_xx_split);
 
-      /* TODO ankitm: use `skip_unsupported_options` for parsing these options too? */
+      /* TODO(@ankitm): use `skip_unsupported_options` for parsing these options too? */
       const int64_t pos_o{str_map_xx_split.first_index_of_try("-o")};
       if (pos_o != -1 && pos_o + 3 < str_map_xx_split.size()) {
         copy_string_to_float({str_map_xx_split[pos_o + 1],



More information about the Bf-blender-cvs mailing list