#96 ESI$Table.Clone, CopyTo iterate too many columns

[Eric Lambart]

Both ESI$Table.Clone and ESI$Table.CopyTo contain this
line:
.For T%C=0:1:I%Column D
But this should read:
.For T%C=0:1:I%Column-1 D

As is, these methods are creating an extra blank column
in the target table.

Furthermore, since these methods clearly have the same
function, they are rather redundant. Presuming one of
them is maintained for backwards compatibility, I suggest
that the older one merely call (delegate to?) the other.

The extra method call may cause a small loss of speed
and efficiency, but currently, CopyTo is already much
less efficient than Clone, as it creates an unnecessary
ESI$List object as an intermediary to copying the table
rows. Clone copies each and every cell directly.

I could have filed a separate bug for that one, but I hope
this is sufficient.