search

ddablib / envvars

Environment Variables Unit
1 stars 2 forks source link

Remove or deprecate TPJEnvVarsEnumerator #8

Open delphidabbler opened 2 years ago

delphidabbler commented 2 years ago

Although TPJEnvVarsEnumerator can be used stand alone, its real purpose is to provide an enumerator for the component. If the component is removed (issue #1) then there is no real need to keep this enumerator class.

Given that TPJEnvironmentVars provides various enumerator methods there is no real need to use TPJEnvVarsEnumerator as a stand alone enumerator, especially since the TPJEnvironmentVars methods are easier to use.

Therefore TPJEnvVarsEnumerator could be deleted, given that it is not currently deprecated, it could be retained but deprecated.

delphidabbler commented 2 years ago

I'm tending to favour removal since leaving the enumerator class in place is redundant given the available enumerator methods of TPJEnvironmentVars .