Skip to content

Commit ae10bff

Browse files
nvmkurucpixar-oss
authored andcommitted
Remove _CRT_SECURE_NO_WARNINGS from testArchUtil
`_CRT_SECURE_NO_WARNINGS` is defined in two places on Windows. This yields the following warning when building `testArchUtil`. ``` testArchUtil.cpp(7): warning C4005: '_CRT_SECURE_NO_WARNINGS': macro redefinition testArchUtil.cpp: note: see previous definition of '_CRT_SECURE_NO_WARNINGS' ``` This PR removes the unneeded `_CRT_SECURE_NO_WARNINGS`. It's not clear what warning `_CRT_SECURE_NO_WARNINGS` was being introduced to suppress in `testArchUtil` as disabling both the `#define` and `msvcdefaults.cmake` doesn't yield any additional warnings in `testArchUtil`. Closes #3885 (Internal change: 2387624)
1 parent 7de5538 commit ae10bff

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

pxr/base/arch/testenv/testArchUtil.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@
44
// Licensed under the terms set forth in the LICENSE.txt file available at
55
// https://openusd.org/license.
66
//
7-
#define _CRT_SECURE_NO_WARNINGS
8-
97
#include "pxr/pxr.h"
108
#include "pxr/base/arch/testArchUtil.h"
119
#include "pxr/base/arch/attributes.h"

0 commit comments

Comments
 (0)