search

ceylon / ceylon.ast

Apache License 2.0
18 stars 3 forks source link

rename compileX to parseX? #101

Closed lucaswerkmeister closed 8 years ago

lucaswerkmeister commented 8 years ago

I just realized that the compileStringLiteral etc. functions are misnamed, since they only parse the node without doing any other part of compilation. Is it too late to change that?

jvasileff commented 8 years ago

I don't think it's too late. My expectation is that there would be (reasonable) compatibility breaks from time to time anyway, do to the nature of the module.

lucaswerkmeister commented 8 years ago

How about we do this in 1.3.0? That will probably include language changes and therefore be incompatible anyways.

(Of course, we can also supply a command that automatically updates source code from compileX to parseX, to aid upgrading.)

lucaswerkmeister commented 8 years ago

Well, 1.3.0 is coming up fast, so let’s see if we can’t get this done!

lucaswerkmeister commented 8 years ago

By the way, this also reduces .car size by 1.9% ;)