diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-07-17 20:12:29 +0200 |
|---|---|---|
| committer |  Lubomir I. Ivanov <neolit123@gmail.com> | 2018-07-18 02:09:15 +0300 |
| commit | 6d72871942adf03e13436ff9342951543351cea6 (patch) | |
| tree | 495d914a7e0f90bd5e3abef731ec48c7ad014360 /subsurface-mobile-main.cpp | |
| parent | 2c6b1a99af103232d420a09023e9e0e6ccee9084 (diff) | |
| download | subsurface-6d72871942adf03e13436ff9342951543351cea6.tar.gz | |
Cleanups to core/color.[c|hpp]
A number of small cleanups to the color-table:
1) Make the profile_color map of static linkage - it is not
used outside of this file.
2) Remove the third color, which originally was planned for
printing. It was not accessed anywhere.
3) Replace QVector<QColor> by std::array<QColor, 2>. Using a
reference-counted, copy-on-write, dynamic container for static
data seems like overkill. std::array<QColor, 2> has exactly the
same run-time impact as QColor[2], but allows for assignment.
4) Use brace-initialization and remove the unneeded COLOR macro.
5) Remove the fill_profile_color function. Simply use static
initialization.
6) Move #includes from .h to .cpp file.
7) Remove text_render_options(_t), which were not used anywhere.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'subsurface-mobile-main.cpp')
| -rw-r--r-- | subsurface-mobile-main.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/subsurface-mobile-main.cpp b/subsurface-mobile-main.cpp index cd9c9dfa4..91bfd65d9 100644 --- a/subsurface-mobile-main.cpp +++ b/subsurface-mobile-main.cpp @@ -83,7 +83,6 @@ int main(int argc, char **argv) else default_prefs.units = IMPERIAL_units; copy_prefs(&default_prefs, &prefs); - fill_profile_color(); fill_computer_list(); parse_xml_init(); |