Hacker News Reader

The "if (temp) free(temp);" line made me chuckle.

Can I ask why? Is it bad style? (I'm fairly new to C)

I'm late to the party here, but `free` is a no-op on a null pointer. So checking for null is just duplicating the first thing `free` does.