setfacl Recursive Functionality

I've just finished a patch to add recursive functionality to setfacl. Before
I officially submit it, I'd like a few suggestions on how to improve the
patch.

The part I'm worried about involves the #define directive at top. I'm not
sure what ramifications using that define might have. I needed it for my
remove_invalid_inherit() function to work.

Thanks,

Shawn

attached: setfacl.patch (universal diff)