CS 3733 Operating Systems, Spring 2005 Assignment 1 Comments
- This assignment was graded on the basis of 30 points.
- Circled positive numbers on your assignment correspond to the
following list of comments with the indicated point penalty.
- A number that appears more than once indicates a severe instance that
generated multiple point penalties.
- A negative circled number indicates additional points lost.
- On this assignment there is a small penalty for lint-related errors.
- In future assignments, not including lint output, or the presence of
lint warnings that you should eliminate will result in large penalties,
possibly up to half of the total point value of the assignment.
- You need to learn how to use lint. Each main program is linted with all
of the pieces needed to run it.
Numbered comments, point penalties in parentheses:
- Staple the assignment in the upper left corner, not the right corner.
- Print your assignment using a fixed point type such as Courier, not a
proportional type font.
- Do not ignore lint warning that are not on the list of warnings you can
ignore.(-1)
- You must test all cases.
For deserialize, read the last sentence
of the first paragraph of Part 2.
Each of the possible failure modes must be tested.(-1)
- Possible buffer overflows are serious problems.
It is too late to check
for the overflow after you have copied the string into a buffer.(-1)
- Do not assume a maximum size for the strings involved.(-1)
- Source lines should be at most 80 characters in length and must fit on one
line when printed.(-1)
- In show_info you must cast the integral data types before printing.
(-1)
- Do not say you have no lint warnings when you have serious ones.(-1)
- In C, you cannot assume a fixed maximum number of digits in an int or long.
- Don't cast an integral type to a float when printing it out.(-1)
- The deserialize function might not set the name if
it reports and error.
- Do not print in serialize or deserialize.
- Avoid memory leaks.(-1)
- Don't use an array when separate variables are more appropriate.