diff options
| author | Ellie Hermaszewska <ellieh@nvidia.com> | 2023-07-06 03:18:14 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-07-05 15:18:14 -0400 |
| commit | 6063304cb8d73d430e7ef81c62cd9822302fcc19 (patch) | |
| tree | 8174e0d5acab31ce057db7b73e8f94b09d786a46 /source/compiler-core | |
| parent | 93948b564d04eda555bf96025e89853be86cff8a (diff) | |
Squash some warnings (#2956)
* restrict -Wno-assume to clang (gcc does not have this warning)
* Add move where possible
Annoyingly this warns for c++17, but will not be necessary with c++20
* Do not partially initialize struct
* Remove unused variable
* Silence unused var warning
It is actually still referenced from an uninstantiated (for now) template
* Use unused var
---------
Co-authored-by: jsmall-nvidia <jsmall@nvidia.com>
Diffstat (limited to 'source/compiler-core')
| -rw-r--r-- | source/compiler-core/slang-source-embed-util.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/compiler-core/slang-source-embed-util.cpp b/source/compiler-core/slang-source-embed-util.cpp index ec0c005cf..45fd9c4b3 100644 --- a/source/compiler-core/slang-source-embed-util.cpp +++ b/source/compiler-core/slang-source-embed-util.cpp @@ -73,7 +73,7 @@ static bool _isHeaderExtension(const UnownedStringSlice& in) // Assume it's a header, and just use the .h extension StringBuilder buf; buf << path << toSlice(".h"); - return buf; + return std::move(buf); } /* static */SourceEmbedUtil::Style SourceEmbedUtil::getDefaultStyle(const ArtifactDesc& desc) |