From 9e6757f20f34eab731bdc30e17e9c8bb6ee360dc Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Wed, 19 Apr 2023 12:45:47 +1000 Subject: [PATCH] Cleanup: expand on why the extension isn't replaced for blend-file save --- source/blender/windowmanager/intern/wm_files.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/source/blender/windowmanager/intern/wm_files.cc b/source/blender/windowmanager/intern/wm_files.cc index 11ee17793c9..3a8670ce377 100644 --- a/source/blender/windowmanager/intern/wm_files.cc +++ b/source/blender/windowmanager/intern/wm_files.cc @@ -3265,8 +3265,9 @@ static bool wm_save_mainfile_check(bContext * /*C*/, wmOperator *op) char filepath[FILE_MAX]; RNA_string_get(op->ptr, "filepath", filepath); if (!BKE_blendfile_extension_check(filepath)) { - /* some users would prefer BLI_path_extension_replace(), - * we keep getting nitpicking bug reports about this - campbell */ + /* NOTE(@ideasman42): some users would prefer #BLI_path_extension_replace(), + * we have had some nitpicking bug reports about this. + * Always adding the extension as users may use '.' as part of the file-name. */ BLI_path_extension_ensure(filepath, FILE_MAX, ".blend"); RNA_string_set(op->ptr, "filepath", filepath); return true;