Adding some flag-setting functionality to the Node_Impl methods so that its user doesn't have to do it explicitly.

Conflicts:
	document_parser.cpp

Even less warnings

malloc does not initialize the memory to zero, so

sass_context *ctx = sass_new_context()

will create a context with a random value in ctx->output_string.

If sass_free_context(ctx) was called immediately thereafter,
ctx->output_string would have a random value and the result of the
free(ctx->output_string) would be undefined.

In the worst case, this corrupts the heap and the process dies much much later.

Also, free isn't delete and mustn't be called with a NULL pointer.

One warning remains and should be taken care of properly.

In any case: this patch should be reviewed carefully.

- more signed/unsigned
- unused variables
- suggest braces around assignment

Remove signed/unsigned warnings

This commit wasn't tested (for a lack of tests) but was reviewed and should be safe.

#define'd constants should not end with a semicolon. 

Fix prototypes. C wants void (C++ doesn't need that).