From ea7b2deffdc67edce7c31754be206e49bf465000 Mon Sep 17 00:00:00 2001
From: Jian Chen <cjian@microsoft.com>
Date: Thu, 18 May 2023 17:08:05 +0000
Subject: [PATCH] Removing C4090 warning  suppression (#15994)

### Description
Removing C4090 warning  suppression after windows pipelines adapt vs2022


### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
---
 java/src/main/native/ai_onnxruntime_OrtTrainingSession.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/java/src/main/native/ai_onnxruntime_OrtTrainingSession.c b/java/src/main/native/ai_onnxruntime_OrtTrainingSession.c
index 938eb1c944..b3b530a8b1 100644
--- a/java/src/main/native/ai_onnxruntime_OrtTrainingSession.c
+++ b/java/src/main/native/ai_onnxruntime_OrtTrainingSession.c
@@ -666,12 +666,6 @@ JNIEXPORT void JNICALL Java_ai_onnxruntime_OrtTrainingSession_schedulerStep
  * Method:    exportModelForInference
  * Signature: (JJJJLjava/lang/String;[Ljava/lang/String;)V
  */
-#ifdef _MSC_VER
-#pragma warning(push)
-// C4090: 'operation' : different 'modifier' qualifiers
-// Freeing 'outputNames' erroneously triggers this warning, it is fixed in VC 2022 and can be removed when that is the baseline compiler.
-#pragma warning(disable : 4090)
-#endif  // _MSC_VER
 JNIEXPORT void JNICALL Java_ai_onnxruntime_OrtTrainingSession_exportModelForInference
   (JNIEnv * jniEnv, jobject jobj, jlong apiHandle, jlong trainApiHandle, jlong nativeHandle, jstring outputPath, jlong numOutputs, jobjectArray outputNamesArr) {
   (void)jobj;  // Required JNI parameter not needed by functions which don't need to access their host object.
@@ -721,6 +715,3 @@ cleanup_array:
   free(javaOutputStrings);
   free(outputNames);
 }
-#ifdef _MSC_VER
-#pragma warning(pop)
-#endif  // _MSC_VER