Add sbbs_t::reinit_msg_ptrs() - eliminates some copy/pasta

Clarify the JS bbs.reinit_msg_ptrs() method description a bit.

src/sbbs3/data_ovl.cpp

@@ -47,6 +47,14 @@ void sbbs_t::putmsgptrs()
 		errormsg(WHERE, ERR_WRITE, "message pointers", 0);
 }
 
+void sbbs_t::reinit_msg_ptrs()
+{
+	for(uint i = 0; i < cfg.total_subs; ++i) {
+		subscan[i].ptr = subscan[i].sav_ptr;
+		subscan[i].last = subscan[i].sav_last;
+	}
+}
+
 static void ProgressSearchingUsers(void* cbdata, int count, int total)
 {
 	sbbs_t* sbbs = ((sbbs_t*)cbdata);

src/sbbs3/execmsg.cpp

@@ -296,10 +296,7 @@ int sbbs_t::exec_msg(csi_t *csi)
 			new_scan_ptr_cfg();
 			return(0);
 		case CS_MSG_PTRS_REINIT:
-			for(i=0;i<cfg.total_subs;i++) {
-				subscan[i].ptr=subscan[i].sav_ptr;
-				subscan[i].last=subscan[i].sav_last; 
-			}
+			reinit_msg_ptrs();
 			bputs(text[MsgPtrsInitialized]);
 			return(0);
 		case CS_MSG_NEW_SCAN_CFG:

src/sbbs3/js_bbs.cpp

@@ -4060,10 +4060,7 @@ js_msgscan_reinit(JSContext *cx, uintN argc, jsval *arglist)
 	JS_SET_RVAL(cx, arglist, JSVAL_VOID);
 
 	rc=JS_SUSPENDREQUEST(cx);
-	for(uint i=0;i<sbbs->cfg.total_subs;i++) {
-		sbbs->subscan[i].ptr=sbbs->subscan[i].sav_ptr;
-		sbbs->subscan[i].last=sbbs->subscan[i].sav_last;
-	}
+	sbbs->reinit_msg_ptrs();
 	sbbs->bputs(sbbs->text[MsgPtrsInitialized]);
 	JS_RESUMEREQUEST(cx, rc);
 
@@ -4642,7 +4639,7 @@ static jsSyncMethodSpec js_bbs_functions[] = {
 	,310
 	},
 	{"reinit_msg_ptrs",	js_msgscan_reinit,	0,	JSTYPE_VOID,	JSDOCSTR("")
-	,JSDOCSTR("re-initialize new message scan pointers")
+	,JSDOCSTR("re-initialize new message scan pointers to values at logon")
 	,310
 	},
 	{"scan_subs",		js_scansubs,		0,	JSTYPE_VOID,	JSDOCSTR("[mode=<tt>SCAN_NEW</tt>] [,all=<tt>false</tt>]")

src/sbbs3/sbbs.h

@@ -700,6 +700,7 @@ public:
 
 	void	getmsgptrs(void);
 	void	putmsgptrs(void);
+	void	reinit_msg_ptrs(void);
 	void	getusrsubs(void);
 	void	getusrdirs(void);
 	uint	getusrsub(uint subnum);