BasicMarquee: add constructor
authorMax Kellermann <max@musicpd.org>
Fri, 23 Feb 2018 08:55:14 +0000 (09:55 +0100)
committerMax Kellermann <max@musicpd.org>
Fri, 23 Feb 2018 08:55:14 +0000 (09:55 +0100)
src/BasicMarquee.hxx
src/hscroll.hxx
test/run_hscroll.cxx

index 65304f4..937187d 100644 (file)
@@ -27,7 +27,7 @@
  * scrolling.
  */
 class BasicMarquee {
-       const char *separator;
+       const char *const separator;
 
        /**
         * The available screen width (in cells).
@@ -56,9 +56,8 @@ class BasicMarquee {
        unsigned offset;
 
 public:
-       void Init(const char *_separator) {
-               separator = _separator;
-       }
+       BasicMarquee(const char *_separator)
+               :separator(_separator) {}
 
        bool IsDefined() const {
                return text != nullptr;
index 397ef04..eae4db6 100644 (file)
@@ -60,8 +60,7 @@ class hscroll {
 
 public:
        hscroll(WINDOW *_w, const char *_separator)
-               :w(_w) {
-               basic.Init(_separator);
+               :w(_w), basic(_separator) {
        }
 
        bool IsDefined() const {
index 3b644f6..a15109d 100644 (file)
@@ -25,8 +25,7 @@ int main(int argc, char **argv)
        width = atoi(argv[3]);
        count = atoi(argv[4]);
 
-       BasicMarquee hscroll;
-       hscroll.Init(argv[2]);
+       BasicMarquee hscroll(argv[2]);
        hscroll.Set(width, argv[1]);
 
        for (unsigned i = 0; i < count; ++i) {