RE: Explanation: New ARI warning Wed Apr 21 01:54:25 UTC 2010

["Pierre Muller" <pierre.muller@ics-cnrs.unistra.fr>]

  Whoops, I forgot ONE big limitation
that might apply to today's email.

> > 1072a1071
> > > gdb/procfs.c:4090: code: sprintf: Do not use sprintf, instead use
> > xstrprintf
> > gdb/procfs.c:4090:      sprintf (name, ', pi->pid, map->pr_mapname);
> > 1100d1098
> > < gdb/procfs.c:5659: code: sprintf: Do not use sprintf, instead use
> > xstrprintf
> > gdb/procfs.c:5659:      sprintf (name, ', pi->pid, map->pr_mapname);

   I wasn't careful enough:
The first 1072 is with '>' meaning new,
but the second is '<' meaning removal.
  Moreover the two lines are identical which probably means that
this is just due to reordering.

  Thus the total number of 'sprintf' uses did not change
but the reordering of the functions that Joel
made lead to this email.

  My apologies to Joel.

  So this reveals already a limitation
of the script: If you move function around, 
it becomes difficult to realize for the script that this is 
only a move, and no new warning.

  Sorry for my wrong first explanation,


Pierre