Hi,

Die_Another_Day

Your right of course but the code l posted was the 'test' version as
denoted by the macro name.

On testing l only looped 20 times and realised that l needed to turn
the screen updating off. On the final version l have determined the
lastrow with code and also amended the variable type as long to
accomodate the 63000 rows.

Its newsgroups like this that really allow developers to swap ideas and
produce good robust applications for the end user.

Once again many thanks for all your input

Regards

Michael Beckinsale