Wanted to point out a couple of thinks I believe should be cleaned up next time another clean up session comes around (unless I am mistaken and I misunderstood something, in which case I would appreciate someboyd explaining my error):
Both changes are on the platform layer:
-malloc.h is still included (line 33 on win32_handmade.cpp), I think this was added for a call to malloc() that was eventually replaced but it is not being used currently.
-On line 64 in win32_handmade.h WIN32_STATE_FILE_NAME_COUNT is defined as MAX_PATH and used. Since it was mentioned on stream that MAX_PATH is not a shippable thing, shouldn´t this be placed inside an #IF HANDMADE_INTERNAL ? There is a note regarding this on line 98 in win32_handmade.cpp but, "comments are always out of date" and although this one isn´t out of date it is out of place.
Both changes are on the platform layer:
-malloc.h is still included (line 33 on win32_handmade.cpp), I think this was added for a call to malloc() that was eventually replaced but it is not being used currently.
-On line 64 in win32_handmade.h WIN32_STATE_FILE_NAME_COUNT is defined as MAX_PATH and used. Since it was mentioned on stream that MAX_PATH is not a shippable thing, shouldn´t this be placed inside an #IF HANDMADE_INTERNAL ? There is a note regarding this on line 98 in win32_handmade.cpp but, "comments are always out of date" and although this one isn´t out of date it is out of place.