From aad5db9a6ad3a265bd8a0012c9a5e0eace48b3a1 Mon Sep 17 00:00:00 2001 From: Tomasz Sowa Date: Mon, 25 Apr 2022 16:35:14 +0200 Subject: [PATCH] let Functions::Add() returns info whether the functions/controller has been added --- winixd/functions/functions.cpp | 12 +++++++----- winixd/functions/functions.h | 4 ++-- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/winixd/functions/functions.cpp b/winixd/functions/functions.cpp index 483799f..9bf3496 100644 --- a/winixd/functions/functions.cpp +++ b/winixd/functions/functions.cpp @@ -197,28 +197,30 @@ void Functions::SetObjects(FunctionBase * fun) } -void Functions::Add(FunctionBase * fun) +bool Functions::Add(FunctionBase * fun) { if( fun->fun.url.empty() ) { log << log1 << "Functions: skipping a function with an empty url" << logend; - return; + return false; } if( Find(fun->fun.url) ) { log << log1 << "Functions: function " << fun->fun.url << " already exists (skipped)" << logend; - return; + return false; } SetObjects(fun); table[fun->fun.url] = fun; + + return true; } -void Functions::Add(FunctionBase & fun) +bool Functions::Add(FunctionBase & fun) { - Add(&fun); + return Add(&fun); } diff --git a/winixd/functions/functions.h b/winixd/functions/functions.h index 7cfd924..78ec7ce 100644 --- a/winixd/functions/functions.h +++ b/winixd/functions/functions.h @@ -191,8 +191,8 @@ public: void SetUser(Item & item); - void Add(FunctionBase * fun); - void Add(FunctionBase & fun); + bool Add(FunctionBase * fun); + bool Add(FunctionBase & fun); private: