bsdperimeter/pfsense-packages — pull request #232 by phil-davis (2012-03-28 12:30:20+00:00)
Title: Code formatting of pfblocker.inc

While debugging the previous change (duplicate rules) I indented all the code within "if/else" statements and "foreach" loops and added brackets to each of these where there were not already brackets - but I only submitted the minimal code change to fix the bug.
This formatting makes it easier to understand and trace the code when looking at it. I have been running this OK on my 2.1-DEV system, installing, uninstalling, enabling, disabling pfBlocker. The diff looks big because a lot of lines had the indenting changed.
I think this makes the code more maintainable in future - but look and see if you agree.