Incorrect attribution of generics parameter list

[wqmeng]

I find the codes and codes following it can not format correctly. Please have a look.

raise Exception.CreateFmt('Unknown flag "%s"', [IfThen(Length(Args) > 0, Args[0], '')]);

Thank you.

[jgardn3r]

Hi @wqmeng,

Can you please provide a minimum reproducible case?
An easy way is to enter it into the web demo, use the Share Example button, and paste that here.

Thanks

[wqmeng]

I am sorry, Here is the minimum reproducible case that after test. Please have a look.

https://integrated-application-development.github.io/pasfmt/?source=dW5pdCBVbml0MTsKCmludGVyZmFjZQoKaW1wbGVtZW50YXRpb24KCmZ1bmN0aW9uIFRTeW1ib2xSZXNvbHZlclBhcnNlci5DdXJyZW50OiBUVG9rZW47CmJlZ2luCiAgaWYgRlBvcyA8IEZUb2tlbnMuQ291bnQgdGhlbgogICAgUmVzdWx0IDo9IEZUb2tlbnNbRlBvc10KICBlbHNlCiAgICBSZXN1bHQgOj0gVFRva2VuLkNyZWF0ZSh0a0VPRiwgJycsIC0xLCAtMSk7CmVuZDsKCmZ1bmN0aW9uIFRTeW1ib2xSZXNvbHZlclBhcnNlci5QYXJzZUZ1bmN0aW9uQ2FsbDogVFN5bWJvbFJlc29sdmVyTm9kZTsKdmFyCiAgRnVuY05hbWU6IHN0cmluZzsKICBBcmdzOiBUQXJyYXk8c3RyaW5nPjsKICBBcmdDb3VudDogSW50ZWdlcjsKICBGbGFnOiBUU3ltYm9sRmxhZzsKICBJOiBJbnRlZ2VyOwpiZWdpbgogIEZ1bmNOYW1lIDo9IExvd2VyQ2FzZShFeHBlY3RXb3JkKTsKICBFeHBlY3RTeW1ib2woJygnKTsKCiAgU2V0TGVuZ3RoKEFyZ3MsIDApOwogIEFyZ0NvdW50IDo9IDA7CgogIGlmIG5vdCBNYXRjaFN5bWJvbCgnKScpIHRoZW4KICBiZWdpbgogICAgcmVwZWF0CiAgICAgIFNldExlbmd0aChBcmdzLCBBcmdDb3VudCArIDEpOwogICAgICBBcmdzW0FyZ0NvdW50XSA6PSBQYXJzZUFyZ3VtZW50VmFsdWU7CiAgICAgIEluYyhBcmdDb3VudCk7CiAgICB1bnRpbCBub3QgTWF0Y2hTeW1ib2woJywnKTsKCiAgICBFeHBlY3RTeW1ib2woJyknKTsKICBlbmQ7CgogIC8vIC0tLSBhbGwoKSAtLS0KICBpZiBGdW5jTmFtZSA9ICdhbGwnIHRoZW4KICBiZWdpbgogICAgaWYgTGVuZ3RoKEFyZ3MpIDw%2BIDAgdGhlbgogICAgICByYWlzZSBFeGNlcHRpb24uQ3JlYXRlKCdhbGwoKSBkb2VzIG5vdCB0YWtlIGFyZ3VtZW50cycpOwogICAgRXhpdChUQWxsU3ltYm9sc05vZGUuQ3JlYXRlKTsKICBlbmQ7CgogIC8vIC0tLSBzZWN0b3IgLyBjb25jZXB0IC0tLQogIGlmIChGdW5jTmFtZSA9ICdzZWN0b3InKSBvciAoRnVuY05hbWUgPSAnY29uY2VwdCcpIHRoZW4KICBiZWdpbgogICAgaWYgTGVuZ3RoKEFyZ3MpIDw%2BIDEgdGhlbgogICAgICByYWlzZSBFeGNlcHRpb24uQ3JlYXRlRm10KCclcygpIGV4cGVjdHMgMSBhcmd1bWVudCcsIFtGdW5jTmFtZV0pOwoKICAgIEV4aXQoVFNlY3RvclN5bWJvbHNOb2RlLkNyZWF0ZShUcmltKEFyZ3NbMF0pKSk7CiAgZW5kOwoKICAvLyAtLS0gZmxhZyAtLS0KICBpZiBGdW5jTmFtZSA9ICdmbGFnJyB0aGVuCiAgYmVnaW4KICAgIGlmIChMZW5ndGgoQXJncykgPD4gMSkgb3IgKG5vdCBUcnlQYXJzZVN5bWJvbEZsYWcoQXJnc1swXSwgRmxhZykpIHRoZW4KICAgICAgcmFpc2UgRXhjZXB0aW9uLkNyZWF0ZUZtdCgnVW5rbm93biBmbGFnICIlcyInLCBbSWZUaGVuKExlbmd0aChBcmdzKSA%2BIDAsIEFyZ3NbMF0sICcnKV0pOwogICAgRXhpdChURmxhZ1N5bWJvbHNOb2RlLkNyZWF0ZShGbGFnKSk7CiAgZW5kOwogICAgICAvLyAtLS0gc3ltYm9scyAtLS0KICBpZiBGdW5jTmFtZSA9ICdzeW1ib2xzJyB0aGVuCiAgYmVnaW4KICAgIGlmIExlbmd0aChBcmdzKSA9IDEgdGhlbgogICAgICBBcmdzIDo9IFNwbGl0U3RyaW5nKEFyZ3NbMF0sICcsJyk7CiAgICBpZiBMZW5ndGgoQXJncykgPSAwIHRoZW4KICAgICAgcmFpc2UgRXhjZXB0aW9uLkNyZWF0ZSgnc3ltYm9scygpIGV4cGVjdHMgYXQgbGVhc3Qgb25lIHN5bWJvbCcpOwogICAgZm9yIEkgOj0gMCB0byBIaWdoKEFyZ3MpIGRvCiAgICAgIEFyZ3NbSV0gOj0gVHJpbShBcmdzW0ldKTsKICAgIEV4aXQoVEV4cGxpY2l0U3ltYm9sc05vZGUuQ3JlYXRlKEFyZ3MpKTsKICBlbmQ7CiAgcmFpc2UgRXhjZXB0aW9uLkNyZWF0ZUZtdCgnVW5rbm93biBzeW1ib2wgcmVzb2x2ZXIgZnVuY3Rpb24gIiVzIicsIFtGdW5jTmFtZV0pOwplbmQ7CgpmdW5jdGlvbiBUU3ltYm9sUmVzb2x2ZXJQYXJzZXIuUGFyc2VGdW5jdGlvbkNhbGxPbGQ6IFRTeW1ib2xSZXNvbHZlck5vZGU7CnZhcgogIEZ1bmNOYW1lOiBzdHJpbmc7CiAgQXJnczogVEFycmF5PHN0cmluZz47CiAgSSwgQXJnQ291bnQ6IEludGVnZXI7CiAgRmxhZzogVFN5bWJvbEZsYWc7CmJlZ2luCiAgRnVuY05hbWUgOj0gTG93ZXJDYXNlKEV4cGVjdFdvcmQpOwogIEV4cGVjdFN5bWJvbCgnKCcpOwoKICBTZXRMZW5ndGgoQXJncywgMCk7CiAgQXJnQ291bnQgOj0gMDsKICBpZiBub3QgTWF0Y2hTeW1ib2woJyknKSB0aGVuCiAgYmVnaW4KICAgIHJlcGVhdAogICAgICBTZXRMZW5ndGgoQXJncywgQXJnQ291bnQgKyAxKTsKICAgICAgQXJnc1tBcmdDb3VudF0gOj0gUGFyc2VBcmd1bWVudFZhbHVlOwogICAgICBJbmMoQXJnQ291bnQpOwogICAgdW50aWwgbm90IE1hdGNoU3ltYm9sKCcsJyk7CiAgICBFeHBlY3RTeW1ib2woJyknKTsKICBlbmQ7CgogIGlmIEZ1bmNOYW1lID0gJ2FsbCcgdGhlbgogIGJlZ2luCiAgICBpZiBMZW5ndGgoQXJncykgPD4gMCB0aGVuCiAgICAgIHJhaXNlIEV4Y2VwdGlvbi5DcmVhdGUoJ2FsbCgpIGRvZXMgbm90IHRha2UgYXJndW1lbnRzJyk7CiAgICBFeGl0KFRBbGxTeW1ib2xzTm9kZS5DcmVhdGUpOwogIGVuZDsKCiAgaWYgU2FtZVRleHQoRnVuY05hbWUsICdzZWN0b3InKSBvciBTYW1lVGV4dChGdW5jTmFtZSwgJ2NvbmNlcHQnKSB0aGVuCiAgYmVnaW4KICAgIGlmIExlbmd0aChBcmdzKSA8PiAxIHRoZW4KICAgICAgcmFpc2UgRXhjZXB0aW9uLkNyZWF0ZUZtdCgnJXMoKSBleHBlY3RzIDEgYXJndW1lbnQnLCBbRnVuY05hbWVdKTsKICAgIEV4aXQoVFNlY3RvclN5bWJvbHNOb2RlLkNyZWF0ZShVcHBlckNhc2UoVHJpbShBcmdzWzBdKSkpKTsKICBlbmQ7CgogIGlmIFNhbWVUZXh0KEZ1bmNOYW1lLCAnZmxhZycpIHRoZW4KICBiZWdpbgogICAgaWYgKExlbmd0aChBcmdzKSA8PiAxKSBvciAobm90IFRyeVBhcnNlU3ltYm9sRmxhZyhBcmdzWzBdLCBGbGFnKSkgdGhlbgogICAgYmVnaW4KICAgICAgcmFpc2UgRXhjZXB0aW9uLkNyZWF0ZUZtdCgnVW5rbm93biBmbGFnICIlcyInLCBbSWZUaGVuKExlbmd0aChBcmdzKSA%2BIDAsIEFyZ3NbMF0sICcnKV0pOwogICAgZW5kOwogICAgRXhpdChURmxhZ1N5bWJvbHNOb2RlLkNyZWF0ZShGbGFnKSk7CiAgZW5kOwogIGlmIFNhbWVUZXh0KEZ1bmNOYW1lLCAnc3ltYm9scycpIHRoZW4KICBiZWdpbgogICAgaWYgTGVuZ3RoKEFyZ3MpID0gMSB0aGVuCiAgICBiZWdpbgogICAgICBBcmdzIDo9IFNwbGl0U3RyaW5nKEFyZ3NbMF0sICcsJyk7CiAgICAgIGZvciBJIDo9IDAgdG8gSGlnaChBcmdzKSBkbwogICAgICAgIEFyZ3NbSV0gOj0gVHJpbShBcmdzW0ldKTsKICAgIGVuZAogICAgZWxzZSBpZiBMZW5ndGgoQXJncykgPSAwIHRoZW4KICAgICAgcmFpc2UgRXhjZXB0aW9uLkNyZWF0ZSgnc3ltYm9scygpIGV4cGVjdHMgYXQgbGVhc3Qgb25lIHN5bWJvbCcpOwogICAgRXhpdChURXhwbGljaXRTeW1ib2xzTm9kZS5DcmVhdGUoQXJncykpOwogIGVuZDsKICByYWlzZSBFeGNlcHRpb24uQ3JlYXRlRm10KCdVbmtub3duIHN5bWJvbCByZXNvbHZlciBmdW5jdGlvbiAiJXMiIGF0IHBvc2l0aW9uICVkJywgW0Z1bmNOYW1lLCBDdXJyZW50LkxpbmVdKTsKZW5kOwoKZW5kLg%3D%3D&settings=d3JhcF9jb2x1bW4gPSAxMjAKYmVnaW5fc3R5bGUgPSAiYXV0byIKZm9ybWF0X211bHRpbGluZV9zdHJpbmdzID0gdHJ1ZQplbmNvZGluZyA9ICJuYXRpdmUiCnVzZV90YWJzID0gZmFsc2UKdGFiX3dpZHRoID0gMgpjb250aW51YXRpb25faW5kZW50cyA9IDIKbGluZV9lbmRpbmcgPSAibmF0aXZlIgo%3D&version=djAuNy4w

[jgardn3r]

This issue is created when a generic parameter list is incorrectly recognised.

In this example, between the < and > are incorrectly deemed to be a generic argument list. This reaps havoc on the following code.

In @wqmeng's example, there are even more lines and tokens between the < and > that should have ended the logical line.

I will look into this. Thanks for the report.

[wqmeng]

@jgardn3r

Thank you for looking into this. Nice job.