PLCgen: location pointers have names ending in '.'
WARNING: Renaming "M2." to "M2_1".
Is this correct? The name is not "M2.", but "M2". Why does it need a rename? It didn't before.
The name generator joins good parts of a name by underscores. The
.
is not a good character, and thus gets removed. OnlyM2
remains as good part, and it doesn't need a_
as there is only one part, so there is nothing to join.
M2
is now a name to avoid for S7-300/S7-400, since it overlaps with address identifiers.I think the
M2.
comes from introducing location pointers for automata. We opt for""
as location pointer variable name. Then we get for an automaton namedM2
,M2.
as location pointer name. Through PLC name generation, the.
disappears. Now, since it renames, we see the.
in the warning message. That is indeed not so nice. It would be nicer if it indicatesWARNING: Renaming "M2" to "M2_1".
.
Addresses #679