stop loading quest template to reset items

`reset_items` it called on start up, and it loads the quest template
file to initialize the items to whatever is in there. Except there is
no item section in the classic module's template file, and the module
system is dead weight at this point, so let's stop loading this thing
for no reason.

Benchmarks:
  hyperfine './zelda -test modules/classic/classic_1st.qst 0 119'

Baseline (classic_1st.qst):
  Time (mean ± σ):      4.239 s ±  0.145 s    [User: 0.926 s, System: 0.410 s]
  Range (min … max):    4.133 s …  4.638 s    10 runs

Don't load quest template (classic_1st.qst):
  Time (mean ± σ):      3.852 s ±  0.094 s    [User: 0.710 s, System: 0.403 s]
  Range (min … max):    3.776 s …  4.107 s    10 runs

Author: connorjclark

src/qst.cpp

@@ -1148,16 +1148,12 @@ void init_spritelists()
 
 bool reset_items(bool validate, zquestheader *Header)
 {
-    bool ret = init_section(Header, ID_ITEMS, NULL, NULL, validate);
+    // qst_dat (classic_qst.dat) has no ID_ITEMS, and the quest template file is a somewhat expensive thing to load
+    // for no reason, so let's stop doing that.
+    // https://discord.com/channels/876899628556091432/876908472728453161/1111405574899376239
+    // bool ret = init_section(Header, ID_ITEMS, NULL, NULL, validate);
+    bool ret = true;
 
-    //Ignore this, but don't remove it
-    /*
-    if (ret)
-      for(int32_t i=0; i<MAXITEMS; i++)
-      {
-        reset_itembuf(&itemsbuf[i], i);
-      }
-    */
     for(int32_t i=0; i<MAXITEMS; i++) reset_itemname(i);
 
     return ret;

src/zc/zc_sys.cpp

@@ -829,20 +829,9 @@ void null_quest()
 	loadquest(qstdat_string,&QHeader,&QMisc,tunes+ZC_MIDI_COUNT,false,true,true,true,skip_flags,0,false);
 }
 
+// TODO remove
 void init_NES_mode()
 {
-	/*
-	// qst.dat may not load correctly without this...
-	QHeader.templatepath[0]='\0';
-	
-	if(!init_colordata(true, &QHeader, &QMisc))
-	{
-		return;
-	}
-	
-	loadfullpal();
-	init_tiles(false, &QHeader);
-	*/
 	null_quest();
 }