That's the correct way to do it. Make sure you put all those brushes in using block(s) or explicitly call Dispose() on each one before you exit your method - unless you are caching them in private fields, in which case you need to make your class IDisposable and implement the Dispose pattern.
Dave
Binging is like googling, it just feels dirtier. Please take your VB.NET out of our nice case sensitive forum. Astonish us. Be exceptional. (Pete O'Hanlon)
BTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)