ucla-cs130 / caviar-hacker-grid Goto Github PK

The implicit rule should not point to a .cc, and instead to a .o (or no extension). Otherwise, this will lead to bugs regarding the dependencies not being able to track tests properly.

If the request is malformed and has no headers (i.e. GET / HTTP/1.1\r\n\r\n), the code will segfault when it tries to access the request data.

Webserver options never gets deleted. The destructor should delete the internal maps.

Currently the Makefile loads files by looking at a list manually entered in the Makefile. There should be a way to make one of two improvements:

1. Automatically infer the test file (instead of having a second list). Naming conventions support that the test file is named the same as the .cc file with a _test appended to the name (%.cc => %_test.cc)
2. Automatically infer all the files, and differentiate the flags run when doing tests. This is much more complex, put should be doable. It would recursively look for all files with the .cc ending, check if it has _test and use the appropriate flags correspondingly. (Or, have two implicit rules with string matching _test.cc and .cc, and understand precedence in Makefiles)

In the Makefile, the target be
`gcov:CPPFLAGS+=-fprofile-arcs -ftest-coverage
instead of hard coding it.

To reproduce:
Make the config port 65536, for example.

http://stackoverflow.com/a/14186634

Special characters are not parsed into their actual expected character -- for example %20 should become a space character.

Currently data isn't streaming -- serving files is loaded into static memory (bad idea 1, heap is better than stack) all at once, and then put to the socket.
Instead, it should read some and write some data, then read some more data and write some more data.

Should only load util-classes as well as the relevant test file -- not all the test files.

It randomly fails (due to a race condition perhaps? needs investigation) and says that ps aux fails with an error code of 1.
Restarting the build can resolve this.