Hi Fernando,

>1) Needs a clenup (I mean a very BIG one, with Beautify and the like :)

For you maybe, I'm comfortable with the style we've been building over the years


>2) Some parts of the code are prone to C0000005 errors, because you are using RETURNs in the middle of a WITH..ENDWITH (this is an old known one. Others can tell you)

Come on, we may have this in test routines, not in production code


>3) You are so Javascript influenced, that you terminate many LOCALs with ";", when VFP only use this as continuation line and not as ending line, like Javascript (you are tempting the luck :)

having one line per variable or parameter is very convenient for reading, adding or reordering
requires a blank line afterwards:

lparameters ;
  tcWarning as String; && {en} warning, empty for a new line {fr} Avertissement, vide pour nouvelle ligne
, lAppendNot as Boolean; && [.F.] {en} do not log warning {fr} Ne pas ajouter l'avertissement au compte-rendu
, lCallingAddNot as Boolean; && [.F.] {en} do not add calling program name as header {fr} ne pas ajouter le nom du programme appelant en-tête

local llResult as Boolean;
, llWarnNew;
, lcWarning;
, lcCallingModule;

personal preference