Cleanup: spelling in comments (make check_spelling_*)

scripts/modules/_bpy_internal/assets/remote_library_listing/asset_downloader.py

@@ -55,7 +55,7 @@ def download_asset_file(
 
     :param save_to: the path on disk where to download to. While the download is
         pending, ".part" will be appended to the filename. When the download
-        finishes succesfully, it is renamed to the final path.
+        finishes successfully, it is renamed to the final path.
     """
     try:
         downloader = _asset_downloaders[asset_library_url]
@@ -107,7 +107,7 @@ def download_preview(
 
     :param dst_filepath: the path on disk where to download to. While the
         download is pending, ".part" will be appended to the filename. When the
-        download finishes succesfully, it is renamed to the final path.
+        download finishes successfully, it is renamed to the final path.
     """
     import time
 

scripts/modules/_bpy_internal/assets/remote_library_listing/cli_listing_downloader.py

@@ -18,7 +18,7 @@ logger = logging.getLogger(__name__)
 
 @dataclasses.dataclass
 class CLIArguments:
-    """Parsed commandline arguments."""
+    """Parsed command-line arguments."""
 
     url: str
 

scripts/modules/_bpy_internal/assets/remote_library_listing/http_metadata.py

@@ -22,7 +22,7 @@ class ExtraFileMetadataProvider(http_dl.MetadataProvider):
 
     The downloader will get the request to download to `file-unsafe.json`.
     However, if `file.json` is still fresh (i.e. the HTTP metadata for the URL
-    is appliccable to that file), the downloader should be able to do a
+    is applicable to that file), the downloader should be able to do a
     conditional download (instead of an unconditional one).
 
     This is implemented as a wrapper for any other MetadataProvider, rather than

scripts/modules/_bpy_internal/assets/remote_library_listing/sync_mutex.py

@@ -31,7 +31,7 @@ def mutex_lock(local_library_path: Path) -> bool:
         - https://www.pythontutorials.net/blog/make-sure-only-a-single-instance-of-a-program-is-running/
         - https://yakking.branchable.com/posts/procrun-2-pidfiles/
 
-    :returns: true if the lock was created succesfully, false if some other
+    :returns: true if the lock was created successfully, false if some other
         Blender already locked this library.
     """
     global _registered_atexit

source/blender/asset_system/AS_remote_library.hh

@@ -173,7 +173,7 @@ class RemoteLibraryLoadingStatus {
    * given timeout duration. Changes the status to failure in that case.
    *
    * Note that this function doesn't do more than check if the timeout is reached, and changing
-   * state to failure if so. It's meant to be called in regular, short intervalls to make the whole
+   * state to failure if so. It's meant to be called in regular, short intervals to make the whole
    * timeout handling work. Current remote asset library loading takes care of this.
    *
    * \return True if the loading status switched to #Status::Failure due to timing out.
@@ -181,7 +181,7 @@ class RemoteLibraryLoadingStatus {
   static bool handle_timeout(StringRef url);
 
  private:
-  /** Update the last update time point, effectively resetting the timout timer. */
+  /** Update the last update time point, effectively resetting the time-out timer. */
   void reset_timeout();
 };
 

source/blender/asset_system/intern/library_types/remote_library.cc

@@ -88,12 +88,12 @@ void RemoteAssetLibrary::refresh_catalogs()
  * \{ */
 
 /*
- * Note: Some of the status setters here only modify status if the current status is
+ * NOTE: Some of the status setters here only modify status if the current status is
  * #RemoteLibraryLoadingStatus::Loading. That is done to avoid status changes after loading ended,
  * mostly after a timeout. E.g. after a timeout of the C++ status because Python didn't send status
  * updates, Python might eventually resume sending updates. These shouldn't affect the C++ status
  * anymore, since the earlier failure aborted the C++ side asset library loading. Plus, the UI
- * might show an error, and would suddenly switch to showing an incompletly loaded asset library.
+ * might show an error, and would suddenly switch to showing an incompletely loaded asset library.
  */
 
 using UrlToLibraryStatusMap = Map<std::string /*url*/, asset_system::RemoteLibraryLoadingStatus>;

source/blender/blenlib/tests/BLI_delaunay_2d_test.cc

@@ -1982,7 +1982,7 @@ template<typename T> void nonzero_winding_negative_only_test()
 
 /**
  * Stress test: overlapping rectangles with shared collinear edge segment.
- * Tests winding when one face's edge is a subsegment of another's edge.
+ * Tests winding when one face's edge is a sub-segment of another's edge.
  *
  * \code{.unparsed}
  * Geometry:

source/blender/bmesh/intern/bmesh_interp.cc

@@ -921,7 +921,7 @@ void BM_data_layer_free(BMesh *bm, CustomData *data, int type)
   data->pool = nullptr;
 
   const bool had_layer = CustomData_free_layer_active(data, eCustomDataType(type));
-  /* Assert because its expensive to realloc - better not do if layer isn't present. */
+  /* Assert because its expensive to reallocate - better not do if layer isn't present. */
   BLI_assert(had_layer != false);
   UNUSED_VARS_NDEBUG(had_layer);
 

source/blender/draw/intern/draw_resource.hh

@@ -66,7 +66,7 @@ inline void ObjectInfos::sync()
 {
   object_attrs_len = 0;
   object_attrs_offset = 0;
-  /* Zero-Initialize since this data might still be accesible (See #154105). */
+  /* Zero-Initialize since this data might still be accessible (See #154105). */
   orco_add = float3(0.0f);
   orco_mul = float3(0.0f);
   ob_color = float4(0.0f);

source/blender/editors/asset/intern/asset_index.hh

@@ -57,7 +57,7 @@ template<typename T = std::monostate> class ReadingResult {
   std::optional<T> success_value;
 
   /**
-   * Even when an operation was performed succesfully, there could have been
+   * Even when an operation was performed successfully, there could have been
    * warnings. These are only intended to be used on success status; on failure,
    * only `failure_reason` is expected to be set. On cancellation, no reason
    * needs to be given (as it is in response to the user cancelling the

source/blender/editors/gpencil_legacy/editaction_gpencil.cc

@@ -416,7 +416,7 @@ bool ED_gpencil_anim_copybuf_paste(bAnimContext *ac, const short offset_mode)
   ANIM_animdata_filter(
       ac, &anim_data, eAnimFilter_Flags(filter), ac->data, eAnimCont_Types(ac->datatype));
   if (BLI_listbase_is_empty(&anim_data)) {
-    /* If no cannels are selected at all, make even unselected layers "targets" for pasting. */
+    /* If no channels are selected at all, make even unselected layers "targets" for pasting. */
     filter = (ANIMFILTER_DATA_VISIBLE | ANIMFILTER_LIST_VISIBLE | ANIMFILTER_NODUPLIS |
               ANIMFILTER_FOREDIT);
     ANIM_animdata_filter(

source/blender/editors/interface/interface_handlers.cc

@@ -5083,8 +5083,8 @@ static int ui_do_but_TEX(
     else if (ELEM(event->type, WHEELUPMOUSE, WHEELDOWNMOUSE) && (event->modifier & KM_CTRL)) {
       if (but->type == ButtonType::SearchMenu) {
         /* Disable value cycling for search buttons. This causes issues because the search data is
-         * moved to the "afterfuncs", but search updating requires it again or somethimes this
-         * event can be triguered twice in row without the button being refreshed. See #147539 and
+         * moved to the `afterfuncs`, but search updating requires it again or sometimes this
+         * event can be triggered twice in row without the button being refreshed. See #147539 and
          * #152976. */
       }
       else {

source/blender/editors/sculpt_paint/mesh/sculpt_pose.cc

@@ -1521,7 +1521,7 @@ static std::unique_ptr<IKChain> ik_chain_init_face_sets_bmesh(Object &object,
 
     if (!next_segment_data) {
       /* It is possible that when traversing neighbors that we no longer have any vertices that
-       * have not been assigned to a face set when trying to find the next segement's starting
+       * have not been assigned to a face set when trying to find the next segment's starting
        * point. All further segments are invalid in this case. */
       break;
     }

source/blender/editors/space_file/file_ops.cc

@@ -2968,7 +2968,7 @@ void file_directory_enter_handle(bContext *C, void * /*arg_unused*/, void *arg_b
 
   if (filelist_is_dir(sfile->files, params->dir)) {
     if (!STREQ(params->dir, filelist_dir(sfile->files))) {
-      /* Keep focus to allow Tab autocompletion. #150689. */
+      /* Keep focus to allow Tab auto-completion. #150689. */
       blender::ui::Button *but = static_cast<blender::ui::Button *>(arg_but);
       textbutton_activate_but(C, but);
       button_clear_selection(but);

source/blender/gpu/intern/gpu_shader_dead_code_elimination.hh

@@ -141,7 +141,7 @@ struct DeadCodeEliminator
     Token name_tok = prev(parenthesis_tok);
     /* WATCH(fclem): It could be that a line directive is put between the return type and the
      * function name (which would mess up the). This is currently not happening with the
-     * current codebase but might in the future. Checking for it would be quite expensive. */
+     * current code-base but might in the future. Checking for it would be quite expensive. */
     if (name_tok != TokenType::Word) {
       return;
     }
@@ -323,8 +323,8 @@ struct DeadCodeEliminator
 
   static StringRef str(const Token t)
   {
-    /* Note: Whitespaces where not merged (because of TokenizePreprocessor), so using
-     * str_view_with_whitespace will be faster.  */
+    /* NOTE: White-spaces where not merged (because of #TokenizePreprocessor),
+     * so using #str_view_with_whitespace will be faster. */
     return t.str_view_with_whitespace();
   }
 

source/blender/gpu/shader_tool/lexit/lexit.hh

@@ -25,17 +25,17 @@ namespace lexit {
  */
 class TokenBuffer {
  protected:
-  /* Input string. */
+  /** Input string. */
   const uint8_t *str_;
-  /* Length of the input string without including null terminator. */
+  /** Length of the input string without including null terminator. */
   const uint32_t str_len_;
-  /* Type of each token. */
+  /** Type of each token. */
   TokenType *types_;
-  /* Starting character index of each token. */
+  /** Starting character index of each token. */
   uint32_t *offsets_;
-  /* Original character index of each next token before whitespace merging (optional). */
+  /** Original character index of each next token before white-space merging (optional). */
   uint32_t *original_offsets_;
-  /* Amount of tokens inside the buffer excluding the terminating EndOfFile token. */
+  /** Amount of tokens inside the buffer excluding the terminating EndOfFile token. */
   uint32_t size_;
 
  public:

source/blender/gpu/vulkan/vk_backend.cc

@@ -502,7 +502,7 @@ void VKBackend::detect_workarounds(VKDevice &device)
    *
    * TODO: We should re-validate vertex input dynamic state as there are multiple vendors with
    * similar issues. It might be an oversight. Will wait for feedback from the driver developers
-   * and perfrom some out of bounds error checks.
+   * and perform some out of bounds error checks.
    */
   if (GPU_type_matches(GPU_DEVICE_QUALCOMM, GPU_OS_WIN, GPU_DRIVER_ANY)) {
     extensions.vertex_input_dynamic_state = false;

source/blender/windowmanager/WM_types.hh

@@ -464,14 +464,17 @@ struct wmNotifier {
 /* NC_OBJECT Object. */
 #define ND_TRANSFORM (18 << 16)
 #define ND_OB_SHADING (19 << 16)
-/** For non-structural posemode changes like transforms. Note: renaming, selecting, bone
- * collections have their own dedicated notifiers, also see #ND_BONE_SELECT. */
+/**
+ * For non-structural pose-mode changes like transforms.
+ * \note renaming, selecting, bone collections have their own dedicated notifiers,
+ * also see #ND_BONE_SELECT.
+ */
 #define ND_POSE (20 << 16)
 #define ND_BONE_ACTIVE (21 << 16)
 /** Intended for selection and visibility changes in pose/armature edit modes.
- * Historically this was also used for most editmode changes (also "structural" like adding,
- * deleting, subdividing, filling, ..., bones). Also covers hiding/revealing (in posemode and
- * editmode). Note this causes a full (possibly slow) rebuild of the Outliner tree. For such
+ * Historically this was also used for most edit-mode changes (also "structural" like adding,
+ * deleting, subdividing, filling, ..., bones). Also covers hiding/revealing (in pose-mode and
+ * edit-mode). Note this causes a full (possibly slow) rebuild of the Outliner tree. For such
  * changes, new code should use #ND_ARMATURE_STRUCTURE. */
 #define ND_BONE_SELECT (22 << 16)
 /** Indicate a change to the structure of the armature; this has implications for both the armature
@@ -479,7 +482,7 @@ struct wmNotifier {
  *
  * The value is set to #ND_BONE_SELECT as a transitional state, as currently that notifier is
  * already used to signify such structural changes. In the future, those uses of #ND_BONE_SELECT
- * should be replaced with #ND_ARMATURE_STRUCTURE, making the selction notifier only relevant for
+ * should be replaced with #ND_ARMATURE_STRUCTURE, making the selection notifier only relevant for
  * selection again. See #153774. */
 #define ND_ARMATURE_STRUCTURE ND_BONE_SELECT
 #define ND_DRAW (23 << 16)

source/blender/windowmanager/intern/wm_surface.cc

@@ -79,7 +79,7 @@ void wm_surface_clear_drawable()
 
     g_drawable = nullptr;
 
-    /* Workaround: For surface drawing, the Userdef runtime DPI/pixelsize values are set to
+    /* Workaround: For surface drawing, the #UserDef runtime DPI/pixel-size values are set to
      * base constants in #wm_surface_constant_dpi_set_userpref called in #wm_surface_make_drawable.
      * This does not affect window rendering as #WM_window_dpi_set_userdef is called in
      * #wm_window_make_drawable. However, some handlers called before window re-draw (such as

tools/check_source/check_spelling_config.py

@@ -922,6 +922,9 @@ files_ignore = {
     "source/blender/blenlib/intern/fnmatch.c",
     "source/blender/draw/intern/shaders/common_fxaa_lib.glsl",
     "source/blender/gpu/shaders/common/gpu_shader_smaa_lib.glsl",
+
+    # Contains `Lorem Ipsum`.
+    "source/blender/blenlib/tests/BLI_resource_strings.h",
 }
 
 # These contain many typos that could be resolved, then removed from this list.