use 16-bit ints for ping times (saves 778 bytes).

1 files changed, 22 insertions, 11 deletions

diff --git a/src/irc.c b/src/irc.c
index f5c0149..8740e15 100644
--- a/src/irc.c
+++ b/src/irc.c
@@ -81,21 +81,31 @@ static void do_chan_nick(void) {
 }
 
 static void do_priv_nick(void) {
-	scr_print_active("*");
-	scr_print_active(msg_src);
-	scr_print_active("* ");
-	bell();
+	if(msg_src) {
+		scr_print_active("*");
+		scr_print_active(msg_src);
+		scr_print_active("* ");
+		bell();
+	}
 }
 
+/* this ping calculation relies on the assumption that
+   nobody's ping time will ever be more than 9 minutes. anyone
+   that lagged will have been disconnected from the server already.
+   if this assumption turns out to be false, the ping time displayed
+   will be wrong (module ~9 mins). I don't think it's ever going to be
+   a real problem.
+   Why do it this way? Because using longs instead of ints bloats the
+   code by 778 bytes! */
 static void print_ping_time(char *p) {
-	static long now, pingtime;
-	static int sec, frac;
+	static unsigned int now, pingtime;
+	static unsigned int sec, frac;
 
 	now = read_rtclok();
-	pingtime = atol(p);
+	pingtime = (unsigned int)atoi(p);
 
-	/* correct for rtclock rollover (every 77 hours) */
-	if(now < pingtime) now |= 0x01000000L;
+	/* correct for rtclock rollover (every ~9 mins) */
+	if(now < pingtime) now |= 0x8000;
 
 	pingtime = now - pingtime;
 
@@ -114,9 +124,10 @@ static void print_ping_time(char *p) {
 	scr_print_active(numbuf);
 	scr_print_active(" sec");
 
-	/* for debugging:
+	/*
+		// for debugging:
 	scr_print_active(" ");
-	ltoa(pingtime, numbuf, 10);
+	itoa(pingtime, numbuf, 10);
 	scr_print_active(numbuf);
 	*/
 }