Auto upper standard data types

[kdorsel]

Twincat will automatically capitalize types and keywords. This will do that for data types.

[Roald87]

Thanks for the PR!

Doesn't TwinCAT automatically do this already? At least in my editor it already does this. See https://infosys.beckhoff.com/content/1033/tc3_plc_intro/9007201787974027.html?id=5455052691128461949 option Convert keywords to uppercase automatically (AutoFormat)

[kdorsel]

Yes, 99% of the time keywords are uppercase automatically. There are a few instances in which they do not though.

For example is you type up the variable type and then click into the body editor without changing lines in the variable definition it will stay lowercase. If the file is then saved it will never be uppercased automatically.

I understand it's rare, but I still think it's something that we should make sure of.

[Roald87]

Can you

1. Also make it pass the following tests:

[InlineData(
    "METHOD Read    : pointer to array[1..nNumber] of int    ",
    "METHOD Read : POINTER TO ARRAY[1..nNumber] OF INT"
)]
[InlineData(
    "METHOD Read    : String(nInt)    ",
    "METHOD Read : STRING(nInt)"
)] 

2. Merge the current merge conflict?

[kdorsel]

Complete rewrite of this upper function. Now includes variable definition as well as objects and uses a static class for better performance.

Took the keyword list from here and added missing data types.
https://infosys.beckhoff.com/english.php?content=../content/1033/tc3_plc_intro/2529971595.html&id=

[Roald87]

Very unfortunate that TwinCAT auto capitalize, converts the time units to upper case. The one with small case units is much better to read, because of the size difference of numbers and small case letters in most fonts. Since TwinCAT does this, I guess we'll stick with this.