changes to client idle time counting

1. Remove leaveClientIdle (unused)
2. s/leaveClientActive/leaveClientIdle/
3. make ISON a leaveClientIdle command (some clients send it periodically
   if a /msg window is left open)

irc/client.go

@@ -332,12 +332,6 @@ func (client *Client) Active() {
 	client.atime = time.Now()
 }
 
-// Touch marks the client as alive (as it it has a connection to us and we
-// can receive messages from it).
-func (client *Client) Touch() {
-	client.idletimer.Touch()
-}
-
 // Ping sends the client a PING message.
 func (client *Client) Ping() {
 	client.Send(nil, "", "PING", client.nick)

irc/commands.go

@@ -12,13 +12,12 @@ import (
 
 // Command represents a command accepted from a client.
 type Command struct {
-	handler           func(server *Server, client *Client, msg ircmsg.IrcMessage, rb *ResponseBuffer) bool
+	handler         func(server *Server, client *Client, msg ircmsg.IrcMessage, rb *ResponseBuffer) bool
-	oper              bool
+	oper            bool
-	usablePreReg      bool
+	usablePreReg    bool
-	leaveClientActive bool // if true, leaves the client active time alone. reversed because we can't default a struct element to True
+	leaveClientIdle bool // if true, leaves the client active time alone
-	leaveClientIdle   bool
+	minParams       int
-	minParams         int
+	capabs          []string
-	capabs            []string
 }
 
 // Run runs this command with the given client/message.
@@ -54,11 +53,10 @@ func (cmd *Command) Run(server *Server, client *Client, msg ircmsg.IrcMessage) b
 		server.tryRegister(client)
 	}
 
-	if !cmd.leaveClientIdle {
+	// most servers do this only for PING/PONG, but we'll do it for any command:
-		client.Touch()
+	client.idletimer.Touch()
-	}
 
-	if !cmd.leaveClientActive {
+	if !cmd.leaveClientIdle {
 		client.Active()
 	}
 
@@ -118,8 +116,9 @@ func init() {
 			minParams: 2,
 		},
 		"ISON": {
-			handler:   isonHandler,
+			handler:         isonHandler,
-			minParams: 1,
+			minParams:       1,
+			leaveClientIdle: true,
 		},
 		"JOIN": {
 			handler:   joinHandler,
@@ -200,16 +199,16 @@ func init() {
 			minParams:    1,
 		},
 		"PING": {
-			handler:           pingHandler,
+			handler:         pingHandler,
-			usablePreReg:      true,
+			usablePreReg:    true,
-			minParams:         1,
+			minParams:       1,
-			leaveClientActive: true,
+			leaveClientIdle: true,
 		},
 		"PONG": {
-			handler:           pongHandler,
+			handler:         pongHandler,
-			usablePreReg:      true,
+			usablePreReg:    true,
-			minParams:         1,
+			minParams:       1,
-			leaveClientActive: true,
+			leaveClientIdle: true,
 		},
 		"PRIVMSG": {
 			handler:   privmsgHandler,