>>The GO TOP is stupid since the scan will start at the top.

Yep, just as SCAN will reselect the correct alias() without the developer having to do it explicitly before ENDSCAN. I've seen both of these redundant practices more than once...

		IF m.wScanForEndOfDay = 1
				REPLACE OutForEOD WITH 0 for OutForEOD<>0 in ScanGrL
			endif
			SCAN
				wMessBody = m.wMessBody + CHR(13) + ALLTRIM(STR(OrdNmb))
			ENDSCAN
			wnAnswer = MESSAGEBOX(m.wMessBody, 4 + 32 + 256, m.goApp.cCaption)

Makes sense to split out the replace, though fwiw you could probably leave out the "in ScanGrL" since it must be already selected.

Other issue would be that the original always replaces OutForEOD with 0 if m.wScanForEndOfDay = 1, which might intentionally affect (for example) a timestamp field or trigger, in which case your version might be better with replace all...

None of that is a snipe obviously. I think we were both there for the mdot flame wars when CamelCase was the answer and mdot was obsolete... till it wasn't. Every so often that happens again in one of these threads and I don't want to miss out!