There is nothing wrong with the patch. What is wrong is the RemovalPolicySettings class which does not have a constructor and leaves the object uninitialized. This then crashes when parsing as the parsing routines tries to free the old values (garbage).
I have had exactly this problem on Linux for the last few days - squid-3 was loading, trying to parse the config, and then aborting out. After that latest commit by Henrik to structs.h, squid-3 now loads up just fine...
Thanks, Reuben
