[Lazarus] Code Observer notes

[Paul Ishenin]

Alexander Klenin wrote:

> This is true, but unrelated.
> First, naming a string constant does not make it translatable,
> it should be made resourcestring (so perhaps it should be a separate
> observation --
> "message is not translatable", where "message" is defined as string
> constant containing at least one word)
> Second, there are different means of i18n, and some of them, like dxgettext,
> do not depend on resourcestrings.
> Third, this same argument can be applied to Fromat, write/writeln and
> other functions already present on ignore list.
> 

1. Later sring and numeric consts will be split between 2 parent nodes.
2. LCLStrConsts and LazarusIDEStrConsts unit uses resourcestrings and 
therefore this is default way to translate strings.

And about WriteLn, Write - usually we are using them in IDE for debug 
purposes only. But I agree that they should not be present in this list 
by default.

On the other handle, this list is adjustable so just feel your own list 
with desired functions :)

Best regards,
Paul Ishenin.