Hacker News Reader

What issue are you taking with the code? Line breaks in the condition?

Presumably the last if statement in the diff.

There aren’t {} brackets after the if

Are you saying they should always be present? Or only when the condition takes multiple lines; i.e. do you take issue with the ifs in zone_name_pref too?

Personally I think the indentation does a good enough job here.

Yes I’m saying they should always be present. It is known to otherwise be error prone, which is why new c-like languages do not permit this (eg go) nor do many style guides (https://google.github.io/styleguide/cppguide.html#Formatting...)

Reply on news.ycombinator.com