cdpxe/WendzelNNTPd

Drop Windows support fully

deavmi opened this issue ยท 7 comments

I added to the todo list text file with some small improvements that could be made.

I saw that there was references to Win32 in the source code. Seeing that Windows is no longer supported (God forbid such a terrible platform for programming) should those references be removed then?

cdpxe commented

Okay. Just please be careful. Like in the OpenBSD project, we should double-check each other's commits from now on.

Ah. I wasn't planning on doing any code changes (as I am still busy learning C when I can (in first year but I like reading ahead in my own time)).

Was just pointing it out. However, when I do become better I shall follow those guidelines. OpenBSD is secure for those purposes.

cdpxe commented

Oh, alright, then I better do the coding as mistakes are quite easy to make and I know the code quite well :)

Yeah, especially since this is C (lots can go wrong very fast from my slight experience I have had with it) - although that is what I love about it (not the going wrong I guess but the flexibility).

I like the way yoi have implemented the database abstraction via a library in the simple way you did. Is pretty cool.

cdpxe commented

thx.

cdpxe commented

Closing issue b/c the Win32 code does not really hurt but removing it might lead to new bugs if done not careful enough. For this reason, I will close this issue.