bindings: cxx: don't shift raw pointers around

There are two leftover instances where we return raw C pointers from
local functions. For safety and consistency with the rest of the code
convert those functions to return unique_ptr.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

diff --git a/bindings/cxx/edge-event-buffer.cpp b/bindings/cxx/edge-event-buffer.cpp
index 3c551dfdc86f05abaf211727f241f9a1e1c0a774..c7347a228550ff062ef9353ebe5eb9ecdeeda085 100644 (file)
--- a/bindings/cxx/edge-event-buffer.cpp
+++ b/bindings/cxx/edge-event-buffer.cpp
@@ -11,9 +11,9 @@ namespace gpiod {
 
 namespace {
 
-::gpiod_edge_event_buffer* make_edge_event_buffer(unsigned int capacity)
+edge_event_buffer_ptr make_edge_event_buffer(unsigned int capacity)
 {
-       ::gpiod_edge_event_buffer* buffer = ::gpiod_edge_event_buffer_new(capacity);
+       edge_event_buffer_ptr buffer(::gpiod_edge_event_buffer_new(capacity));
        if (!buffer)
                throw_from_errno("unable to allocate the edge event buffer");
 

diff --git a/bindings/cxx/line-config.cpp b/bindings/cxx/line-config.cpp
index 0ab54f49303eeb18afe77c233c057a3039253b22..1179e0a7af163797d4a8504cf7459533c9188d31 100644 (file)
--- a/bindings/cxx/line-config.cpp
+++ b/bindings/cxx/line-config.cpp
@@ -14,9 +14,9 @@ namespace gpiod {
 
 namespace {
 
-::gpiod_line_config* make_line_config()
+line_config_ptr make_line_config()
 {
-       ::gpiod_line_config *config = ::gpiod_line_config_new();
+       line_config_ptr config(::gpiod_line_config_new());
        if (!config)
                throw_from_errno("Unable to allocate the line config object");