Chained calls can result in extra parentheses in macro code if the first call has arguments

[syeopite]

Bug Report

I'm not entirely sure if this is a bug but the following fails to compile because dig("key").itself gets rendered as (dig("key")).itself with a set of extra parentheses around the first call:

a_container = {} of String => String

macro access(container, through_expression)
    {{container}}.{{through_expression.stringify.id}}
end

access a_container,
through_expression: dig("key").itself

[straight-shoota]

Yes, this looks very much like a bug. There should be no reason to have these parentheses there.

It looks like ToSVisitor#need_parens is too broad here.