well, the !! were originaly from you. i thought comment =~ regexp returns the matched regexp content and the !! are hacking it to booleans :). Because of the = if they are probably unnecessary.
obs = if !!(comment =~~ OBSOLETE_REGEXP) should assing true or false to obs (the result of the if clause) and obsolete? will always return obs. therefor it should return true/false.