[Asterisk-code-review] core: Config and XML tweaks needed for geolocation (asterisk[development/16/geolocation])

[George Joseph]

Attention is currently required from: Kevin Harwell.
George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/18067 )

Change subject: core: Config and XML tweaks needed for geolocation
......................................................................


Patch Set 3:

(3 comments)

File include/asterisk/config.h:

https://gerrit.asterisk.org/c/asterisk/+/18067/comment/7d4075e1_0665045c 
PS3, Line 1005: struct ast_variable *head
> I think this can be made "const" since "head" is (and should) not being modified.
Done


https://gerrit.asterisk.org/c/asterisk/+/18067/comment/ef11ae9a_748821b8 
PS3, Line 1006: struct ast_str *str
> I worry about the potential for a caller trying to use "str" after it's been invalidated. Why not just pass it in as a pointer to a pointer?
> 
> In doing so you can still check for, and create if NULL. Also, it makes it so there is no chance of a caller using an invalid pointer after this call (well they'd still have to check for NULL).

That's weird.  I though I already did that. Ah well.


File include/asterisk/xml.h:

https://gerrit.asterisk.org/c/asterisk/+/18067/comment/0a292b99_e8c10c62 
PS3, Line 205:  * \brief Set or reset anm element's name.
> s/anm/an
Done



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

Gerrit-Project: asterisk
Gerrit-Branch: development/16/geolocation
Gerrit-Change-Id: I330a5f63dc0c218e0d8dfc0745948d2812141ccb
Gerrit-Change-Number: 18067
Gerrit-PatchSet: 3
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Attention: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Tue, 22 Feb 2022 18:13:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220222/578c3cc2/attachment.html>