- Fixed error display - peeking was pushing recorded `cursor` position out of sync

Sun, 26 Aug 2012 23:18:59 +0300

author
Teemu Piippo <crimsondusk64@gmail.com>
date
Sun, 26 Aug 2012 23:18:59 +0300
changeset 63
0557babc8675
parent 62
824ab7b28e3c
child 64
dc5db6335601

- Fixed error display - peeking was pushing recorded `cursor` position out of sync
- Added a rudimentary dynamic array class.. not used yet.

array.h file | annotate | diff | comparison | revisions
common.h file | annotate | diff | comparison | revisions
databuffer.h file | annotate | diff | comparison | revisions
main.cxx file | annotate | diff | comparison | revisions
scriptreader.cxx file | annotate | diff | comparison | revisions
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/array.h	Sun Aug 26 23:18:59 2012 +0300
@@ -0,0 +1,183 @@
+/*
+ *	botc source code
+ *	Copyright (C) 2012 Santeri `Dusk` Piippo
+ *	All rights reserved.
+ *	
+ *	Redistribution and use in source and binary forms, with or without
+ *	modification, are permitted provided that the following conditions are met:
+ *	
+ *	1. Redistributions of source code must retain the above copyright notice,
+ *	   this list of conditions and the following disclaimer.
+ *	2. Redistributions in binary form must reproduce the above copyright notice,
+ *	   this list of conditions and the following disclaimer in the documentation
+ *	   and/or other materials provided with the distribution.
+ *	3. Neither the name of the developer nor the names of its contributors may
+ *	   be used to endorse or promote products derived from this software without
+ *	   specific prior written permission.
+ *	4. Redistributions in any form must be accompanied by information on how to
+ *	   obtain complete source code for the software and any accompanying
+ *	   software that uses the software. The source code must either be included
+ *	   in the distribution or be available for no more than the cost of
+ *	   distribution plus a nominal fee, and must be freely redistributable
+ *	   under reasonable conditions. For an executable file, complete source
+ *	   code means the source code for all modules it contains. It does not
+ *	   include source code for modules or files that typically accompany the
+ *	   major components of the operating system on which the executable file
+ *	   runs.
+ *	
+ *	THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ *	AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ *	IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ *	ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
+ *	LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ *	CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ *	SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ *	INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ *	CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ *	ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ *	POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include <stdlib.h>
+#include <stdio.h>
+#include <string.h>
+#include "common.h"
+
+#define ITERATE_SUBSCRIPTS(link) \
+	for (link = data; link; link = link->next)
+
+// Single element of an array
+template <class T> class arrayElement {
+public:
+	T value;
+	arrayElement<T>* next;
+	
+	arrayElement () {
+		next = NULL;
+	}
+};
+
+// Dynamic array
+template <class T> class array {
+public:
+	array () {
+		data = NULL;
+	}
+	
+	array (T* stuff, unsigned int c) {
+		printf ("%d elements\n", c);
+		data = NULL;
+		
+		for (unsigned int i = 0; i < c; i++)
+			push (stuff[c]);
+	}
+	
+	~array () {
+		if (data)
+			deleteElement (data);
+	}
+	
+	void push (T stuff) {
+		arrayElement<T>* e = new arrayElement<T>;
+		e->value = stuff;
+		e->next = NULL;
+		
+		if (!data) {
+			data = e;
+			return;
+		}
+		
+		arrayElement<T>* link;
+		for (link = data; link && link->next; link = link->next);
+		link->next = e;
+	}
+	
+	T pop () {
+		int pos = size() - 1;
+		if (pos == -1)
+			error ("array::pop: tried to pop an array with no elements\n");
+		T res = subscript (pos);
+		remove (pos);
+		return res;
+	}
+	
+	void remove (unsigned int pos) {
+		if (!data)
+			error ("tried to use remove on an array with no elements");
+		
+		if (pos == 0) {
+			// special case for first element
+			arrayElement<T>* first = data;
+			data = data->next;
+			delete first;
+			return;
+		}
+		
+		arrayElement<T>* link = data;
+		unsigned int x = 0;
+		while (link->next) {
+			if (x == pos - 1)
+				break;
+			link = link->next;
+			x++;
+		}
+		if (!link)
+			error ("no such element in array\n");
+		
+		arrayElement<T>* nextlink = link->next->next;
+		delete link->next;
+		link->next = nextlink;
+	}
+	
+	unsigned int size () {
+		unsigned int x = 0;
+		arrayElement<T>* link;
+		ITERATE_SUBSCRIPTS(link)
+			x++;
+		return x;
+	}
+	
+	T& subscript (unsigned int i) {
+		arrayElement<T>* link;
+		unsigned int x = 0;
+		ITERATE_SUBSCRIPTS(link) {
+			if (x == i)
+				return link->value;
+			x++;
+		}
+		
+		error ("array: tried to access subscript %u in an array of %u elements\n", i, x);
+		return data->value;
+	}
+	
+	T* out () {
+		int s = size();
+		T* out = new T[s];
+		unsigned int x = 0;
+		
+		arrayElement<T>* link;
+		ITERATE_SUBSCRIPTS(link) {
+			out[x] = link->value;
+			x++;
+		}
+		
+		return out;
+	}
+	
+	T& operator [] (const unsigned int i) {
+		return subscript (i);
+	}
+	
+	void operator << (const T stuff) {
+		push (stuff);
+	}
+	
+private:
+	void deleteElement (arrayElement<T>* e) {
+		if (e->next)
+			deleteElement (e->next);
+		delete e;
+	}
+	
+	arrayElement<T>* data;
+};
\ No newline at end of file
--- a/common.h	Sat Aug 25 05:22:32 2012 +0300
+++ b/common.h	Sun Aug 26 23:18:59 2012 +0300
@@ -43,7 +43,6 @@
 
 #include <stdio.h>
 #include <stdarg.h>
-#include <typeinfo> 
 #include <stdint.h>
 #include "bots.h"
 #include "str.h"
@@ -100,16 +99,14 @@
 #endif
 
 // Power function
-template<class T> T pow (T a, int b) {
+template<class T> T pow (T a, unsigned int b) {
 	if (!b)
 		return 1;
 	
 	T r = a;
 	while (b > 1) {
 		b--;
-		
-		// r *= a fails here with large numbers
-		r += a * a;
+		r = r * a;
 	}
 	
 	return r;
@@ -121,7 +118,6 @@
 }
 
 // Byte datatype
-// typedef unsigned long int word;
 typedef int32_t word;
 typedef unsigned char byte;
 
--- a/databuffer.h	Sat Aug 25 05:22:32 2012 +0300
+++ b/databuffer.h	Sun Aug 26 23:18:59 2012 +0300
@@ -215,10 +215,6 @@
 		if (u == MAX_MARKS)
 			error ("mark reference quota exceeded, all goto-statements, if-structs and loops add refs\n");
 		
-		// NOTE: Do not check if the mark actually exists here since a
-		// reference may come in the code earlier than the actual mark
-		// and the new mark number can be predicted.
-		//	11/8/12: eh? The mark is always created first.
 		ScriptMarkReference* r = new ScriptMarkReference;
 		r->num = marknum;
 		r->pos = writesize;
--- a/main.cxx	Sat Aug 25 05:22:32 2012 +0300
+++ b/main.cxx	Sun Aug 26 23:18:59 2012 +0300
@@ -52,13 +52,16 @@
 #include "commands.h"
 #include "stringtable.h"
 #include "variables.h"
+#include "array.h"
 
 #include "bots.h"
 #include "botcommands.h"
 
+// List of keywords
 const char* g_Keywords[] = {
 	"break",
 	"case",
+	"continue",
 	"default",
 	"do",
 	"else",
@@ -76,7 +79,6 @@
 	
 	// These ones aren't implemented yet but I plan to do so, thus they are
 	// reserved. Also serves as a to-do list of sorts for me. >:F
-	"continue",
 	"enum", // Would enum actually be useful? I think so.
 	"func", // Would function support need external support from zandronum?
 	"return",
--- a/scriptreader.cxx	Sat Aug 25 05:22:32 2012 +0300
+++ b/scriptreader.cxx	Sun Aug 26 23:18:59 2012 +0300
@@ -45,6 +45,16 @@
 #include "common.h"
 #include "scriptreader.h"
 
+#define STORE_POSITION \
+	bool _atnewline = atnewline; \
+	unsigned int _curline = curline[fc]; \
+	unsigned int _curchar = curchar[fc];
+
+#define RESTORE_POSITION \
+	atnewline = _atnewline; \
+	curline[fc] = _curline; \
+	curchar[fc] = _curchar;
+
 // ============================================================================
 ScriptReader::ScriptReader (str path) {
 	token = "";
@@ -152,6 +162,7 @@
 char ScriptReader::PeekChar (int offset) {
 	// Store current position
 	long curpos = ftell (fp[fc]);
+	STORE_POSITION
 	
 	// Forward by offset
 	fseek (fp[fc], offset, SEEK_CUR);
@@ -166,6 +177,7 @@
 	
 	// Rewind back
 	fseek (fp[fc], curpos, SEEK_SET);
+	RESTORE_POSITION
 	
 	return c[0];
 }
@@ -267,6 +279,7 @@
 	// Store current information
 	str storedtoken = token;
 	int cpos = ftell (fp[fc]);
+	STORE_POSITION
 	
 	// Advance on the token.
 	while (offset >= 0) {
@@ -281,6 +294,7 @@
 	fseek (fp[fc], cpos, SEEK_SET);
 	pos[fc]--;
 	token = storedtoken;
+	RESTORE_POSITION
 	return tmp;
 }
 
@@ -337,7 +351,7 @@
 // ============================================================================
 void ScriptReader::ParserMessage (const char* header, char* message) {
 	if (fc >= 0 && fc < MAX_FILESTACK)
-		fprintf (stderr, "%sIn file %s, at line %u, col %u: %s\n",
+		fprintf (stderr, "%s%s:%u:%u: %s\n",
 			header, filepath[fc], curline[fc], curchar[fc], message);
 	else
 		fprintf (stderr, "%s%s\n", header, message);

mercurial