Nechci rejpat, svůj účel to splní, ale možná pár dobře míněních rad :
- Pokud se jedná o velkou spoustu emailů, není na škodu využít vlákna, nebo aspoň BackgroundWorkera v podobě - client.SendAsync()
- using u StreamReaderu není potřeba, GC ho nepošle do pryč, protože jej hned vzápětí použije while

Další věci jsou spíše otázka vkusu, třeba -

if (this.checkBox1.Checked)
vs.
if (this.checkBox1.Checked == true)

Attachment item = new Attachment(this.label6.Text);
message.Attachments.Add(item);
vs.
Attachment item = new Attachment(this.label6.Text);
message.Attachments.Add(item);
item.Dispose(); //Pro ulehčení GC

Také bych si vytvořil samotnou třídu pro práci s Attachment, Mailera a nějaký String Parser pro ulehčení práce a pro jistou znovupoužitelnost kódu, ale to už jsou jen mé preference

A jiné, ale každopádně Good Work