From 062fcd00d1943a90ef7c8d50f084989522926fac Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Fri, 8 Aug 2025 06:41:04 +0000 Subject: [PATCH] Cleanup: improve confusing/incomplete code-comments --- source/blender/asset_system/AS_asset_library.hh | 2 +- source/blender/blenkernel/BKE_tracking.h | 2 +- source/blender/editors/sculpt_paint/paint_image_proj.cc | 5 ++++- source/blender/editors/transform/transform.cc | 2 +- source/blender/editors/uvedit/uvedit_rip.cc | 2 +- source/blender/python/intern/bpy_rna.cc | 2 +- source/blender/windowmanager/intern/wm_event_system.cc | 2 +- 7 files changed, 10 insertions(+), 7 deletions(-) diff --git a/source/blender/asset_system/AS_asset_library.hh b/source/blender/asset_system/AS_asset_library.hh index 73fe6a15b08..df5342623c8 100644 --- a/source/blender/asset_system/AS_asset_library.hh +++ b/source/blender/asset_system/AS_asset_library.hh @@ -41,7 +41,7 @@ class AssetRepresentation; class AssetLibrary { eAssetLibraryType library_type_; /** - * The name this asset library will be displayed in the UI as. Will also be used as a weak way + * The name this asset library will be displayed as in the UI. Will also be used as a weak way * to identify an asset library (e.g. by #AssetWeakReference). */ std::string name_; diff --git a/source/blender/blenkernel/BKE_tracking.h b/source/blender/blenkernel/BKE_tracking.h index 632d557e146..93362e414ba 100644 --- a/source/blender/blenkernel/BKE_tracking.h +++ b/source/blender/blenkernel/BKE_tracking.h @@ -213,7 +213,7 @@ bool BKE_tracking_track_has_enabled_marker_at_frame(struct MovieTrackingTrack *t * \note frame number should be in clip space, not scene space. */ typedef enum eTrackClearAction { - /* Clear path from `ref_frame+1` up to the. */ + /* Clear path from `ref_frame+1` up to the current frame. */ TRACK_CLEAR_UPTO, /* Clear path from the beginning up to `ref_frame-1`. */ TRACK_CLEAR_REMAINED, diff --git a/source/blender/editors/sculpt_paint/paint_image_proj.cc b/source/blender/editors/sculpt_paint/paint_image_proj.cc index c16b5df229f..05c6825d412 100644 --- a/source/blender/editors/sculpt_paint/paint_image_proj.cc +++ b/source/blender/editors/sculpt_paint/paint_image_proj.cc @@ -486,7 +486,10 @@ struct ProjPixel { short x_px, y_px; - /** if anyone wants to paint onto more than 65535 images they can bite me. */ + /** + * Use a short to reduce memory use. + * This limits the total number of supported images to 65535 which seems reasonable. + */ ushort image_index; uchar bb_cell_index; diff --git a/source/blender/editors/transform/transform.cc b/source/blender/editors/transform/transform.cc index d63d1b65e71..f2458dc864c 100644 --- a/source/blender/editors/transform/transform.cc +++ b/source/blender/editors/transform/transform.cc @@ -1101,7 +1101,7 @@ wmOperatorStatus transformEvent(TransInfo *t, wmOperator *op, const wmEvent *eve case TFM_MODAL_TRACKBALL: case TFM_MODAL_ROTATE_NORMALS: case TFM_MODAL_VERT_EDGE_SLIDE: - /* Only switch when. */ + /* Only switch when supported. */ if (!transform_mode_is_changeable(t->mode)) { break; } diff --git a/source/blender/editors/uvedit/uvedit_rip.cc b/source/blender/editors/uvedit/uvedit_rip.cc index 3276b292f54..f2a75775795 100644 --- a/source/blender/editors/uvedit/uvedit_rip.cc +++ b/source/blender/editors/uvedit/uvedit_rip.cc @@ -916,7 +916,7 @@ static wmOperatorStatus uv_rip_exec(bContext *C, wmOperator *op) float aspx, aspy; { - /* Note that we only want to run this on the. */ + /* Note that we only want to run this on the active object as this defines the UV image. */ Object *obedit = CTX_data_edit_object(C); ED_uvedit_get_aspect(obedit, &aspx, &aspy); } diff --git a/source/blender/python/intern/bpy_rna.cc b/source/blender/python/intern/bpy_rna.cc index 6181943de9e..a4ae708cb7e 100644 --- a/source/blender/python/intern/bpy_rna.cc +++ b/source/blender/python/intern/bpy_rna.cc @@ -2865,7 +2865,7 @@ static PyObject *pyrna_prop_collection_subscript(BPy_PropertyRNA *self, PyObject return pyrna_prop_collection_subscript_slice(self, start, stop); } if (PyTuple_Check(key)) { - /* Special case, for ID data-blocks we. */ + /* Special case, for ID data-blocks. */ return pyrna_prop_collection_subscript_str_lib_pair( self, key, "bpy_prop_collection[id, lib]", true); } diff --git a/source/blender/windowmanager/intern/wm_event_system.cc b/source/blender/windowmanager/intern/wm_event_system.cc index 212da5c241e..84f8bb8e505 100644 --- a/source/blender/windowmanager/intern/wm_event_system.cc +++ b/source/blender/windowmanager/intern/wm_event_system.cc @@ -4046,7 +4046,7 @@ static eHandlerActionFlag wm_event_do_region_handlers(bContext *C, wmEvent *even CTX_wm_region_set(C, region); - /* Call even on non mouse events, since the. */ + /* Call even on non mouse events, since the handlers may still use this value. */ wm_region_mouse_co(C, event); const wmWindowManager *wm = CTX_wm_manager(C);