Revert "MGCP: fix pattern warning"
For me this change causes MGCP parsing errors:
setverdict(fail): pass -> fail reason: "Could not extract parameters for code "C""
Apparently the \r is after all necessary to parse MGCP. Maybe the '\r' is
implicit when an '\n' occurs, but the non-SDP part of MGCP has *only* '\r' line
breaks.
This reverts commit a9a52fff15
.
Change-Id: I81d105b951590310c67f14f0b5d0c2777e206c5e
This commit is contained in:
parent
a9a52fff15
commit
f6e991c61f
|
@ -54,7 +54,7 @@ module MGCP_Types {
|
||||||
MgcpVersion ver
|
MgcpVersion ver
|
||||||
} with {
|
} with {
|
||||||
variant "SEPARATOR(' ', '[\t ]+')"
|
variant "SEPARATOR(' ', '[\t ]+')"
|
||||||
variant "END('\n', '([\n])|(\n)')"
|
variant "END('\r\n', '([\r\n])|(\r\n)')"
|
||||||
}
|
}
|
||||||
|
|
||||||
type record MgcpParameter {
|
type record MgcpParameter {
|
||||||
|
@ -63,7 +63,7 @@ module MGCP_Types {
|
||||||
} with {
|
} with {
|
||||||
variant "BEGIN('')"
|
variant "BEGIN('')"
|
||||||
variant "SEPARATOR(': ', ':[\t ]+')"
|
variant "SEPARATOR(': ', ':[\t ]+')"
|
||||||
variant "END('\n', '([\n])|(\n)')"
|
variant "END('\r\n', '([\r\n])|(\r\n)')"
|
||||||
}
|
}
|
||||||
|
|
||||||
type set of MgcpParameter MgcpParameterList with {
|
type set of MgcpParameter MgcpParameterList with {
|
||||||
|
@ -76,7 +76,7 @@ module MGCP_Types {
|
||||||
SDP_Message sdp optional
|
SDP_Message sdp optional
|
||||||
} with {
|
} with {
|
||||||
variant "BEGIN('')"
|
variant "BEGIN('')"
|
||||||
variant (sdp) "BEGIN('\n','([\n])|(\n)')"
|
variant (sdp) "BEGIN('\r\n','([\r\n])|(\r\n)')"
|
||||||
}
|
}
|
||||||
|
|
||||||
external function enc_MgcpCommand(in MgcpCommand id) return charstring
|
external function enc_MgcpCommand(in MgcpCommand id) return charstring
|
||||||
|
@ -90,7 +90,7 @@ module MGCP_Types {
|
||||||
charstring string optional
|
charstring string optional
|
||||||
} with {
|
} with {
|
||||||
variant "SEPARATOR(' ', '[\t ]+')"
|
variant "SEPARATOR(' ', '[\t ]+')"
|
||||||
variant "END('\n', '([\n])|(\n)')"
|
variant "END('\r\n', '([\r\n])|(\r\n)')"
|
||||||
}
|
}
|
||||||
|
|
||||||
type record MgcpResponse {
|
type record MgcpResponse {
|
||||||
|
@ -99,7 +99,7 @@ module MGCP_Types {
|
||||||
SDP_Message sdp optional
|
SDP_Message sdp optional
|
||||||
} with {
|
} with {
|
||||||
variant "BEGIN('')"
|
variant "BEGIN('')"
|
||||||
variant (sdp) "BEGIN('\n','([\n])|(\n)')"
|
variant (sdp) "BEGIN('\r\n','([\r\n])|(\r\n)')"
|
||||||
}
|
}
|
||||||
|
|
||||||
external function enc_MgcpResponse(in MgcpResponse id) return charstring
|
external function enc_MgcpResponse(in MgcpResponse id) return charstring
|
||||||
|
|
Loading…
Reference in New Issue