Use a reference instead of a pointer, as it will always be present now.

2 files changed, 2 insertions, 2 deletions

diff --git a/vespalib/src/tests/invokeservice/invokeservice_test.cpp b/vespalib/src/tests/invokeservice/invokeservice_test.cpp
index bfd33cd7cca..4b987839e50 100644
--- a/vespalib/src/tests/invokeservice/invokeservice_test.cpp
+++ b/vespalib/src/tests/invokeservice/invokeservice_test.cpp
@@ -34,7 +34,7 @@ TEST("require that now is moving forward") {
     InvokeCounter a;
     InvokeServiceImpl service(1ms);
     EXPECT_EQUAL(0u, a._count);
-    auto ra = service.registerInvoke([&prev, &a, now=service.nowPtr() ]() noexcept {
+    auto ra = service.registerInvoke([&prev, &a, now= service.nowRef() ]() noexcept {
         EXPECT_GREATER(now->load(), prev);
         prev = now->load();
         a.inc();
diff --git a/vespalib/src/vespa/vespalib/util/invokeserviceimpl.h b/vespalib/src/vespa/vespalib/util/invokeserviceimpl.h
index 2448e381610..beb57ca1ce0 100644
--- a/vespalib/src/vespa/vespalib/util/invokeserviceimpl.h
+++ b/vespalib/src/vespa/vespalib/util/invokeserviceimpl.h
@@ -20,7 +20,7 @@ public:
     InvokeServiceImpl & operator=(const InvokeServiceImpl &) = delete;
     ~InvokeServiceImpl() override;
     std::unique_ptr<IDestructorCallback> registerInvoke(InvokeFunc func) override;
-    const std::atomic<steady_time> * nowPtr() const { return &_now; }
+    const std::atomic<steady_time> & nowRef() const { return _now; }
 private:
     using IdAndFunc = std::pair<uint64_t, InvokeFunc>;
     class Registration;