Unit test project does not build about evil-dicom HOT 2 CLOSED

Anders, I updated the Selection Module and wanted to keep a consistent pattern with element data selection in the core library. I intend to use SingleValue and MultiplicityValue only internally from here on out. Public access to multiplicity or first element data will be the old fashioned :

".Data" for single T
and the new ".Data_" for multiple (List)

the underscore gives you strong typing to multiplicity data. I just felt that the old solution for dealing with multiplicity data was harder and not as terse as it could have been. Multiplicity access is explained @ http://evildicom.rexcardan.com/getting-started/data-multiplicity/

I will update the UnitTests to make sense with the new update.

from evil-dicom.

Many thanks for the explanation and the new commit, now the unit tests build and run as desired.

Quick comment on Data_: it is a rather unconventional naming of a C# property; have you considered naming it for example Datas instead? (English is not my first language, so Datas might not make any sense, but at least for me the code would be more readable without underscores in the property names.)

Best regards,
Anders

from evil-dicom.