Use of map/grep in void context, use foreach instead