if this is really an "external" optional node,
then this must be moved up in the struct definition to match
position with the XXNames[] array, and to match usage via GENnode[]
if it is merly an "internal" node, which might be monitored via CKTfindBranch,
then it should be removed from the XXNames[] array, again to fit usage via GENnode[]
this is a "pin-pointer" commit,
pointing to places for source code verification
we have int Nodes in every device instance,
these are overlaid with struct GENinstance.GENnode[] and will be used
from there too.
the length entries and the XXnames[] arrays *must* fit this.
FIXME, the previous commit might interfere with certain devices having optional ports, which previously where initialised to zero, now are initialised to -1 !
inp2m.c, INP2M(), bug fix, drop excess GENnode initialisation
Thats done in CKTcrtElt() now,
and avoids overwritting private device internal Node variables,
which are expected to be initialised to zero (by TMALLOC)
