If you're going to SET something, then be sure to un-set it (SET db = Nothing) when you're done with it. Any object that you SET to a variable occupies machine memory until something happens to release it. Simple procedures don't even need the additional code if you want to avoid SETing anything, and don't need the Else part if the logic dictates that only one other route will be taken if the first is not true:
Code:
If DCount("[LastName]", "QmatQselBenefitsToTblPermBenefits") = 0 Then
CurrentDb.Execute "INSERT INTO QmatQselBenefitsToTblPermBenefits (LastName) VALUES ('No Additions To Report')", dbFailOnError
MsgBox "No additions to report this month"
End If
MsgBox "These volunteers will be added to the Abenity eligiblity list"
rest of code...
Please use code tags and indentation in your posts for more than a couple of lines.
The more we hear silence, the more we begin to think about our value in this universe.
Paraphrase of Professor Brian Cox.