This is an archive of the discontinued LLVM Phabricator instance.

[windows] Remove unused pybind exception params
ClosedPublic

Authored by NathanielMcVicar on May 30 2022, 10:58 PM.

Details

Summary

Resolve MSVC warning C4104 for unreferenced variable

Diff Detail

Event Timeline

NathanielMcVicar requested review of this revision.May 30 2022, 10:58 PM

Fixed reference, I'm not clear why this isn't const, since unlike other pybind exceptions cast_error doesn't appear to be handled by Python.

mehdi_amini accepted this revision.May 31 2022, 12:40 AM
This revision is now accepted and ready to land.May 31 2022, 12:40 AM
This revision was automatically updated to reflect the committed changes.