prism icon indicating copy to clipboard operation
prism copied to clipboard

Published 20 hours ago • verified publisher icon ruby

Add support for C++

Open egiurleo opened this issue 1 year ago • 1 comments

It is currently impossible to use the Prism C API in C++ projects because it uses the field name operator, which is a reserved word in C++. This PR changes the name of the operator field to operator_ and adds a CI workflow to ensure that we can build C++ files with Prism.

egiurleo avatar May 02 '24 18:05 egiurleo

Thanks @egiurleo! I ended up going a slightly different way because after living with operator_ being different in Ruby vs C it was weirding me out. So I renamed those fields to binary_operator in all layers here: https://github.com/ruby/ruby/pull/10758.

I still would like to merge the github action and the test script though.

kddnewton avatar May 10 '24 14:05 kddnewton

@kddnewton Removed the commit that introduced operator_ and just kept the testing.

egiurleo avatar May 14 '24 19:05 egiurleo