[Asterisk-code-review] Revert "PJSIP_CONTACT: add missing argument documentation" (asterisk[13])

George Joseph asteriskteam at digium.com
Wed Dec 11 10:35:49 CST 2019


George Joseph has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/13397 )

Change subject: Revert "PJSIP_CONTACT: add missing argument documentation"
......................................................................

Revert "PJSIP_CONTACT: add missing argument documentation"

This reverts commit 3778e1abcd9ffb7fb92253b8333e048239d9f348.

Reason for revert: Regression in XML validation.

validity error : Content model of enumlist is not determinist:
(configOptionToEnum | (configOptionToEnum , enum+) | enum+)

As we are preparing to do releases and this is not critical
I am reverting this for now until resolved.

Change-Id: I07b539d0406e8af08934a91223e850444bde1b75
---
M doc/appdocsxml.dtd
M funcs/func_pjsip_contact.c
2 files changed, 1 insertion(+), 7 deletions(-)

Approvals:
  George Joseph: Looks good to me, but someone else must approve; Approved for Submit
  Pascal Cadotte Michaud: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved



diff --git a/doc/appdocsxml.dtd b/doc/appdocsxml.dtd
index 9f00ee5..21f1b9d 100644
--- a/doc/appdocsxml.dtd
+++ b/doc/appdocsxml.dtd
@@ -110,7 +110,7 @@
   <!ATTLIST option implies CDATA "">
   <!ATTLIST option hasparams CDATA "">
 
-  <!ELEMENT enumlist (configOptionToEnum|(configOptionToEnum,enum+)|enum+)>
+  <!ELEMENT enumlist (configOptionToEnum|enum+)>
   <!ELEMENT enum (para|note|warning|parameter|enumlist|info|xi:include)*>
   <!ATTLIST enum name CDATA "">
 
diff --git a/funcs/func_pjsip_contact.c b/funcs/func_pjsip_contact.c
index cb67277..6904df0 100644
--- a/funcs/func_pjsip_contact.c
+++ b/funcs/func_pjsip_contact.c
@@ -62,12 +62,6 @@
 					<configOptionToEnum>
 						<xi:include xpointer="xpointer(/docs/configInfo[@name='res_pjsip']/configFile[@name='pjsip.conf']/configObject[@name='contact']/configOption)"/>
 					</configOptionToEnum>
-					<enum name="rtt">
-						<para>The RTT of the last qualify</para>
-					</enum>
-					<enum name="status">
-						<para>Status of the contact</para>
-					</enum>
 				</enumlist>
 			</parameter>
 		</syntax>

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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: I07b539d0406e8af08934a91223e850444bde1b75
Gerrit-Change-Number: 13397
Gerrit-PatchSet: 1
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Pascal Cadotte Michaud <pcm at wazo.io>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20191211/68e850c6/attachment.html>


More information about the asterisk-code-review mailing list