The problem is most likely that you have not declared the StopNow
variable. If you don't declare the variable, VBA will create it
within the procedure in which it finds it. Therefore, you have
two completely variables named StopNow, one in each procedure.

The solution is to declare a global variable above and outside of
any procedu

Public StopNow As Boolean

If you had an "Option Explicit" statement at the top of the
module, as you should, the cause of the problem would be obvious
because you would get a "Variable Not Defined" compiler error.


--
Cordially,
Chip Pearson
Microsoft MVP - Excel
Pearson Software Consulting, LLC
www.cpearson.com



"grahammal"
wrote in
message
...

Sub Button1_Click()
StopNow = False
For i = 1 To 30000
Range("A1").Value = i
' DoEvents
If StopNow Then Exit For
Next i
End Sub


Sub Button2_Click()
StopNow = True
End Sub

I have tried the above idea to stop a macro continuing to run
but it
dos'nt work for me.
I generated the 2 buttons, copied the relevent code to the
buttons and
pressed the first button.
I can see the value clocking up in cell A1 but you can't press
the
second button to stop the macro from running???
Am I being stupid or have I missed something?


--
grahammal
------------------------------------------------------------------------
grahammal's Profile:
http://www.excelforum.com/member.php...o&userid=20336
View this thread:
http://www.excelforum.com/showthread...hreadid=477826