Skip to content
This repository was archived by the owner on Apr 21, 2025. It is now read-only.

Conversation

@cbracken
Copy link
Member

@cbracken cbracken commented Apr 13, 2022

Suppresses several Windows-specific warnings emitted during swiftshader
builds:

  • On swiftshader builds against Win32 C++ headers, warnings are emitted
    about subclassing std::iterator, which was deprecated in C++17. This
    affects a header file in swiftshader's third-party directory:
    third_party/swiftshader/third_party/llvm-subzero/include/llvm/ADT/iterator.h

  • Three functions in libVulkan.cpp are re-declared with a difference in
    dllexport attribute only.

  • Silence implicit string conversion warnings in two places in
    swiftshader where asserts of the form assert(!"Error msg") are used.

Issue: flutter/flutter#59199

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Suppresses several Windows-specific warnings emitted during swiftshader
builds:

* On swiftshader builds against Win32 C++ headers, warnings are emitted
  about subclassing std::iterator, which was deprecated in C++17. This
  affects a header file in swiftshader's third-party directory:
  third_party/swiftshader/third_party/llvm-subzero/include/llvm/ADT/iterator.h

* Three functions in libVulkan.cpp are re-declared with a difference in
  dllexport attribute only.

* Silence implicit string conversion warnings in two places in
  swiftshader where asserts of the form assert(!"Error msg") are used.

Issue: flutter/flutter#59199
@cbracken cbracken merged commit 60d942d into flutter:master Apr 13, 2022
@cbracken cbracken deleted the iterator-deprecation-warnings branch April 13, 2022 23:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants