diff options
author | Jason Zaugg <jzaugg@gmail.com> | 2015-11-12 15:03:15 +1000 |
---|---|---|
committer | Jason Zaugg <jzaugg@gmail.com> | 2015-11-13 09:21:59 +1000 |
commit | 72478ba994f2f021c87658bfecd4dd0751ea66a2 (patch) | |
tree | 1cebb8a329c65b8432582a04eea71c345420eabd /bincompat-backward.whitelist.conf | |
parent | 44ae563d375ba462dff621d7e1c5abad1da825af (diff) | |
download | scala-72478ba994f2f021c87658bfecd4dd0751ea66a2.tar.gz scala-72478ba994f2f021c87658bfecd4dd0751ea66a2.tar.bz2 scala-72478ba994f2f021c87658bfecd4dd0751ea66a2.zip |
SI-9555 Refactor SystemProperties to avoid mutable map
A mutable map with help text was lazy populated if and when
lazy vals containing `BooleanProp` for a particular system was
initialized.
Reviewing this code, I noticed two problems: the class doesn't appear
to be designed for multithreaded access (e.g. someone could call
`help` while the map is being updated.), and the help method
won't return anything until the vals below are accessed for
the first time.
We don't use the help method anywhere else, but kept it around
in the unlikely event it was being used by some other tool.
I've done away with the mutable map by just moving the help messages
into a match in the help method itself.
Diffstat (limited to 'bincompat-backward.whitelist.conf')
0 files changed, 0 insertions, 0 deletions