Which should probably be extracted to a helper so you can call:
<%= menu_items_for(@menu_pages, '|') %>
And there will probably be someone who says "you should use do-end for a multi-line block", but I'm believing (and trying to follow) Jim Weirich's style of using {} when the value of the block is used and do-end otherwise (or when precedence dictates).