fixing bug - don't use pointer receivers unless necessary

message.go

@@ -14,7 +14,7 @@ type Message struct {
 	ClosesChannel bool
 }
 
-func (m *Message) Clone() Message {
+func (m Message) Clone() Message {
 	results := Message{
 		Dialect:       strings.Clone(m.Dialect),
 		ID:            strings.Clone(m.ID),
@@ -66,13 +66,13 @@ func ByteIsSpace(b byte) bool {
 // Lines that are not encapsulated in XML tags or
 // are not recognized as part of a known dialect
 // are called chatter.
-func (m *Message) Chatter() bool {
+func (m Message) Chatter() bool {
 	return len(m.Dialect) == 0 && len(m.ID) == 0 && len(m.Parameters) == 0
 }
 
 // Lines that have no associated ChannelID are
 // called anonymous messages.
-func (m *Message) Anonymous() bool {
+func (m Message) Anonymous() bool {
 	return !m.Chatter() && len(m.ID) == 0
 }