saymrwulf/onnxruntime
1
0
Fork 0
mirror of https://github.com/saymrwulf/onnxruntime.git synced 2026-05-14 20:48:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Removing C4090 warning suppression (#15994)

Browse source 
### 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. -->
This commit is contained in:
Jian Chen 2023-05-18 17:08:05 +00:00 committed by GitHub
parent 0c815a95b7
commit ea7b2deffd
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
java/src/main/native/ai_onnxruntime_OrtTrainingSession.c
View file

@ -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