@malat @taylorho this looks like it made it back in after the revert, so the merge request can probably be closed. In current master, 85203c30a ("feat: 12-bit allocation support", Taylor's commit) sits above the revert 7d7e82870 in the history, and the tree carries the handling today: gdcmPixelFormat.cxx has "if( BitsAllocated == 12 )", with matching code in gdcmRLECodec.cxx and gdcmJPEGLSCodec.cxx. The MR16BitsAllocated_8BitsStored.dcm case is covered by c7f1dde93. Taylor, in case it was not clear...
@malat as you noted above, this landed in master as 4c35c1f4d, and the SWIG side is in Wrapping/Python/gdcmswig.i too. The merge request itself is still marked open though, so it shows up as pending work. Could it be closed?
This looks like it is already in master and can be closed. @malat commit a44d5819a carries this exact change, same author and date as f37983 here. Verified against current master: gdcmPreamble.h now has Preamble(Preamble const &):Internal(nullptr) calling Create(), and gdcmFileMetaInformation.h has the defaulted copy constructor and assignment operator. So the fix landed, only the merge request was left open. Thanks to Mad Vitaliy for tracking down the uninitialised Internal.
Code cleaning for C++11
ENH: Add SimpleITK <-> ITK image conversion to the Python interface
COMP: Pass /Zc:__cplusplus when compiling SWIG wrappers with MSVC
COMP: Mangle libpng symbols added since the last vendored update
ENH: Configure the ObjectFactory autoload symbol name