Ok, not worth changing/debating, but I'd just like to say that I've seen this style in vellum and occasionally HQ and I don't really like it.  "A block of procedural initialization code"?  The whole CommcareSettings function is "procedural initialization code", even if some of it is defining methods.

Reasons I don't like this:

- extra lines/noise and an extra level of indentation
- if you have one of these blocks because you're worried about overwriting variables defined above in the same scope then it means your function is too long
- kind of similar to $(function () {}); at a glance