StringLiteral.camelcase does not handle (lower: true) parameter

[wealth]

module A
  macro define_method(decl)
    def {{decl.var.camelcase(lower: true)}}
      puts "does not work!"
    end
  end
end

class Z
  include A
  define_method test_column : String
end

Z.new.testColumn

Result:

There was a problem expanding macro 'define_method'

Code in test.cr:11:3

 11 | define_method test_column : String
      ^
Called macro defined in test.cr:2:3

 2 | macro define_method(decl)

Which expanded to:

 > 1 | def TestColumn
                     ^
Error: unexpected token: NEWLINE

Expected: handles lowercase and defines testColumn method

Crystal 0.31.1 [0e2e1d067] (2019-09-30)

LLVM: 8.0.0
Default target: x86_64-unknown-linux-gnu

[Blacksmoke16]

Duplicate of #8292?

[wealth]

yep, sorry for double post

[straight-shoota]

Reduced:

{{"foo_bar".camelcase(lower: true)}} # => "FooBar"

[Blacksmoke16]

Do we want to support the same UX? Currently named_args aren't included in the macro method interpret method. It would be doable but might take a bit of work. If not the syntax would just be like {{"foo_bar".camelcase(true)}} which would be quite a bit easier to implement.

@straight-shoota @asterite thoughts?

[asterite]

It's something nice to have, but I know it's a bit tedious to implement.