shared_ptr_from_python.hpp shadow storage compiler warning

Open
#396 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
45/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
cpp
Domain
compilers

Research direction

Inspect the referenced change and include/boost/python/converter/shared_ptr_from_python.hpp lines 44-65, then reproduce the warning with the MSVS 2022 build context described in the issue. Confirm that the shadow-storage warning is no longer emitted and that the affected shared_ptr conversion still builds successfully.

Written by the indexing model from the issue text.

Description

Please check change to shared_ptr_from_python.hpp from https://github.com/boostorg/python/commit/aca3c80c4f80dd3d52219e1ee299e08bb980bcfd

https://github.com/boostorg/python/blob/47d5bc76f69e20625214381c930a2fad5765e2b3/include/boost/python/converter/shared_ptr_from_python.hpp#L44-L65

C:\projects\pythonscript\packages\boost.1.80.0\lib\native\include\boost\python\converter\shared_ptr_from_python.hpp(52,1): warning C4456: declaration of 'storage' hides previous local declaration (compiling source file ..\src\NotepadPython.cpp) [C:\projects\pythonscript\PythonScript\project\PythonScript2010.vcxproj]
C:\projects\pythonscript\packages\boost.1.80.0\lib\native\include\boost\python\converter\shared_ptr_from_python.hpp(46,17): message : see declaration of 'storage' (compiling source file ..\src\NotepadPython.cpp) [C:\projects\pythonscript\PythonScript\project\PythonScript2010.vcxproj]
C:\projects\pythonscript\packages\boost.1.80.0\lib\native\include\boost\python\converter\shared_ptr_from_python.hpp(45): message : while compiling class template member function 'void boost::python::converter::shared_ptr_from_python<T,std::shared_ptr>::construct(PyObject *,boost::python::converter::rvalue_from_python_stage1_data *)' [C:\projects\pythonscript\PythonScript\project\PythonScript2010.vcxproj]

From https://ci.appveyor.com/project/bruderstein/pythonscript/builds/44940991/job/g43lxe0hqbj7lrhx build with MS VS2022.

Looks like a merge issue.

Dominant language
C++
Stars
537
Forks
223
Avg merge
11h 22m
Merged PRs (30d)
2

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from boostorg/python

All issues in boostorg/python

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.