From: Simon Marchi <simon.marchi@efficios.com>
Date: Tue, 26 Mar 2024 20:45:47 +0000 (+0000)
Subject: Fix: use `OptionalBorrowedObject::hasObject()` in `CommonOptionField::hasField()`
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=a804026d23ba45312488cd6595cdcbfb5031584d;p=babeltrace.git

Fix: use `OptionalBorrowedObject::hasObject()` in `CommonOptionField::hasField()`

This is needed since `OptionalBorrowedObject::operator bool()` is
explicit.

Change-Id: Iabd899be72607b1d5eb1a42dec919c14d0e33868
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Reviewed-on: https://review.lttng.org/c/babeltrace/+/12714
Reviewed-by: Philippe Proulx <eeppeliteloop@gmail.com>
---

diff --git a/src/cpp-common/bt2/field.hpp b/src/cpp-common/bt2/field.hpp
index aca49d49..a75a054e 100644
--- a/src/cpp-common/bt2/field.hpp
+++ b/src/cpp-common/bt2/field.hpp
@@ -1597,7 +1597,7 @@ public:
 
     bool hasField() const noexcept
     {
-        return this->field();
+        return this->field().hasObject();
     }
 
     OptionalBorrowedObject<CommonField<LibObjT>> field() const noexcept