if only I know which one is right :) gcc is a little picky when it accepts attributes out side of declarations, let me just verify this works before I sync the two prs