Sven Schüle wrote:Hope that helps a little bit, although the "hash error" messages at startup time do still appear after fixing this.
The messages go away if I set "*use_hashtables 1" in "dabbaba.ini".
I tried to provide an ini file containing only that single line, which crashed in ReadINIFile(). Changing NULL into "" (empty string) in line 888 helped, so I recommend that you fix this.
The code is very difficult to read for me for at least these reasons:
- chaotic indentation policy (indeed, no policy)
- too much unused code left within comments (I'd better remove it, while archiving old versions)
- sometimes too many empty lines, sometimes not enough empty lines
- too often many statements on the same line (o.k., others do this too sometimes but it should be kept on an acceptable level)
- frequent use of goto, often instead of simple for loop
- use of macros like "#define equal ==" makes life hard for C programmers ...
So if your intention is to improve your program then you most probably want to reduce the number of bugs in your code. One requirement to achieve this is that you can read and understand your code. Of course the fact that I have difficulties reading it does not mean that you have, too, but from what you wrote I guess there are at least some parts that you don't fully understand.
Therefore I want to encourage you to improve readability. The code is not bad at all but all I want to say is that there is much potential for improvement.
Sven