05-11-2008, 08:41 PM
One way to improve this is remove the useless dims
Dim x As Long
Dim y As Long
in first sub
Dim i As Long
in the second sub. and third, and fourth
I suppose the Temp Rec is a bit of a waste as well, you only use the vars you input there once, might as well just copy over that data.
Dimming both offsets public and using them only once sounds like a waste as well.
So seems the code is good, but can be cleared up a bit.
Dim x As Long
Dim y As Long
in first sub
Dim i As Long
in the second sub. and third, and fourth
I suppose the Temp Rec is a bit of a waste as well, you only use the vars you input there once, might as well just copy over that data.
Dimming both offsets public and using them only once sounds like a waste as well.
So seems the code is good, but can be cleared up a bit.