[Asterisk-code-review] db: Notify user if deleted DB entry didn't exist. (asterisk[master])

N A asteriskteam at digium.com
Thu Apr 28 11:44:23 CDT 2022


Attention is currently required from: Joshua Colp.
N A has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/18361 )

Change subject: db: Notify user if deleted DB entry didn't exist.
......................................................................


Patch Set 2:

(1 comment)

Patchset:

PS2: 
> Is there anything preventing this from being done by adding new API calls that actually provide deta […]
What new API call would you envision here? Or do you mean making ast_db_del implicitly check that it exists before deleting, and returning -1 if it doesn't as opposed to 0 as it does now? (Or a "stricter" version of ast_db_del that does this?)



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/18361
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Ic84e3eddcd66c7a6ed7fea91cdfd402568378b18
Gerrit-Change-Number: 18361
Gerrit-PatchSet: 2
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: Joshua Colp <jcolp at sangoma.com>
Gerrit-Comment-Date: Thu, 28 Apr 2022 16:44:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220428/216aac02/attachment.html>


More information about the asterisk-code-review mailing list