Open oliverjanik opened 8 years ago
This would probably require the C# driver to re-implement some of the logic currently done in the Java driver's python code. That's where the mapping from reql terms to method names happens. I think RethinkDb.Driver uses only the output of that process, java_term_info.json
.
For reference, the mapping of keywords happens here: https://github.com/rethinkdb/rethinkdb/blob/next/drivers/java/metajava.py#L143
You could easily modify metajava.py to have an empty JAVA_KEYWORDS
array
Echoing everything the masterful @deontologician said.
Just a small addition: We do a tiny itsy-bitsy bit processing on top of java_term_info.json
in CSharpTermInfoMutator.cs
as a safety and sanity check. IIRC, there was one or two keywords that caused the C# compiler to freak out.
The C# driver still needs the Java's do_
because the C# driver has a temporary and internal Java API compatibility layer that forwards Java naming AST calls to the primary C# naming AST. This internal layer is needed for maintaining compatibility with the Java unit tests. Also see: https://github.com/bchavez/RethinkDb.Driver/issues/18#issuecomment-171535799 [paragraph Regarding No.1]
Here's an example:
public Funcall Do_ ( Javascript js )
{
Arguments arguments = new Arguments();
arguments.CoerceAndAdd(js);
return new Funcall (arguments);
}
internal Funcall do_ ( Javascript js )
{
return Do_ ( js );
}
I think writing code to maintain special cases for Do
and do_
would add complexity at the moment. Also, the benefit of keeping compatibility with Java's unit tests is too huge for us to remove right now.
Eventually, we'll be able to remove the internal Java API compatibility layer soon as (IIRC) https://github.com/rethinkdb/rethinkdb/issues/5003 is done. Once https://github.com/rethinkdb/rethinkdb/issues/5003 is done, I think we'll use a csharp_coverter.py
to generate C# unit tests using @deontologician 's multireql tool. Once we have these two items, we can go back and remove the Do_
do_
:hankey: and just win with Do
:trophy: . The internal Java API compatibility layer won't be needed anymore.
do_ method in java only has underscore because it conflicts with the keyword, right? With the capitalization C# does not have this probelm