From: Scott Klein <kleinmatic@gmail.com>
Date: Sat, 1 Aug 2026 10:44:01 +0000 (+0000)
Subject: patch 9.2.0889: VMS: spurious "INVALID DECC FEATURE VALUE" message at every startup
X-Git-Tag: v9.2.0889^0
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=52593298a256f50918b64ee8b5a7fc154dc3422a;p=thirdparty%2Fvim.git

patch 9.2.0889: VMS: spurious "INVALID DECC FEATURE VALUE" message at every startup

Problem:  On OpenVMS, Vim prints "INVALID DECC FEATURE VALUE" once per
          DECC feature at every startup whenever that feature is already
          set to the value Vim wants -- e.g. when DECC$ARGV_PARSE_STYLE,
          DECC$EFS_CASE_PRESERVE or DECC$EFS_CHARSET are defined as
          logical names. Conversely, a genuinely out-of-range value is
          silently ignored, so the diagnostic fires exactly when it
          should not and never when it should (after v9.1.1591)
Solution: Add braces so that the "else" binds to the range check rather
          than to the inner "if", matching the existing indentation and
          the evident intent (Scott Klein).

related: #17810
closes:  #20904

Supported by AI.

Signed-off-by: Scott Klein <kleinmatic@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
---

diff --git a/src/os_vms.c b/src/os_vms.c
index 17fa067743..3c82299394 100644
--- a/src/os_vms.c
+++ b/src/os_vms.c
@@ -1005,9 +1005,11 @@ vms_init(void)
 	    feat_value_max = decc$feature_get_value(feat_index, 3);
 
 	    if ((decc_feat_array[i].value >= feat_value_min) && (decc_feat_array[i].value <= feat_value_max))
+	    {
 		// Valid value.  Set it if necessary
 		if (feat_value != decc_feat_array[i].value)
 		    sts = decc$feature_set_value(feat_index, 1, decc_feat_array[i].value);
+	    }
 	    else
 		// Invalid DECC feature value
 		printf("INVALID DECC FEATURE VALUE, %d: %d <= %s <= %d.\n",
diff --git a/src/version.c b/src/version.c
index 0fdd61f59f..dcec47b106 100644
--- a/src/version.c
+++ b/src/version.c
@@ -758,6 +758,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    889,
 /**/
     888,
 /**/