[Bug]: Incomplete error message on xml:fromString method

[SasinduDilshara]

Description

import ballerina/io;
public function main() {
    string a = string `
        <?xml version='1.0' encoding='UTF-8'?>
            <soap:Envelope xmlns:xsd="http://www.w3.org/2001/XMLSchema" 
            xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
            xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/" 
            xmlns:tns="http://www.example.com/">;
                <soap:Body>
            <tns:receive.cdiResponse>
            <Response>
            <Code>1000</Code>
            <Description><![CDATA[OK]]></Description>
            </Response>
            </tns:receive.cdiResponse>
                </soap:Body>
            </soap:Envelope>
        `;
    
    xml|error fromString = xml:fromString(a);
    io:println(fromString);
}

in here the program outputs,

error("failed to parse xml: Illegal processing instruction target ("xml"); xml (case insensitive) is reserved by the specs.
 at [row,col {unknown-source}]: [2,13]")

This error message is not clear for the user. It is better to have a meaningful error message.

Affected version - 2201.8.0

Steps to Reproduce

No response

Affected Version(s)

No response

OS, DB, other environment details and versions

No response

Related area

-> Compilation

Related issue(s) (optional)

No response

Suggested label(s) (optional)

No response

Suggested assignee(s) (optional)

No response

[Nadeeshan96]

Here the error message is clear and correct. According to ballerina-platform/ballerina-spec#1225 (comment), we do not allow "xml" as a processing instruction target when parsing xml strings. Hence we cannot include the xml declaration also. If we change the processing instruction target to another value, say abc, it works without any problem.

[warunalakshitha]

As per above comment I am closing the issue. Please reopen if you have further queries on this.