diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-02-28 23:37:09 +0100 |
|---|---|---|
| committer |  Lubomir I. Ivanov <neolit123@gmail.com> | 2018-03-14 13:55:36 +0200 |
| commit | d1572a8d9574e80955796b8667317c8dbd376081 (patch) | |
| tree | ab58f9920a20b5c55bd54474f2edcfdc4eb2b904 /core/prefs-macros.h | |
| parent | b72cc1f31784346884c8552c22f1e4c3c6d5ed1d (diff) | |
| download | subsurface-d1572a8d9574e80955796b8667317c8dbd376081.tar.gz | |
Cleanup: introduce copy_qstring() function
strdup(qPrintable(s)) and copy_string(qPrintable(s)) were such common
occurrences that they seem worthy of a short helper-function.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/prefs-macros.h')
| -rw-r--r-- | core/prefs-macros.h | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/core/prefs-macros.h b/core/prefs-macros.h index 16a92aa24..72d5c60b2 100644 --- a/core/prefs-macros.h +++ b/core/prefs-macros.h @@ -2,6 +2,8 @@ #ifndef PREFSMACROS_H #define PREFSMACROS_H +#include "core/qthelper.h" + #define SB(V, B) s.setValue(V, (int)(B->isChecked() ? 1 : 0)) #define GET_UNIT(name, field, f, t) \ @@ -60,11 +62,11 @@ else \ prefs.field = defval -#define GET_TXT(name, field) \ - v = s.value(QString(name)); \ - if (v.isValid()) \ - prefs.field = strdup(qPrintable(v.toString())); \ - else \ +#define GET_TXT(name, field) \ + v = s.value(QString(name)); \ + if (v.isValid()) \ + prefs.field = copy_qstring(v.toString()); \ + else \ prefs.field = copy_string(default_prefs.field) #define SAVE_OR_REMOVE_SPECIAL(_setting, _default, _compare, _value) \ |