git rid of c-bindings artifacts

Author: iliabylich

README.md

@@ -57,7 +57,7 @@ All node classes fully match node structs of the original Rust implementation. Y
 + Clone the repo
 + Set environment variables:
     + `TARGET` (e.g. `export TARGET=x86_64-apple-darwin`, no default value)
-    + `CC` (e.g. `gcc`)
+    + `CXX` (e.g. `g++`)
     + `BUILD_ENV` (`debug` or `release`, `debug` is the default value)
 + run `make tests/run` to run tests
 + run `make libruby_parser_cpp.a` (or `make libruby_parser_cpp.lib` for MSVC) to get a static library
@@ -66,8 +66,8 @@ All node classes fully match node structs of the original Rust implementation. Y
 # Structure
 
 + Each directory has its own `build.mk` file that is included my the main `Makefile`
-+ Rust parser with basic C bindings is located under `ruby-parser-c`.
-+ Actual C bindings are located in `*.{h, c}` files in the root directory
++ Rust parser with basic C++ bindings is located under `ruby-parser-cpp`.
++ Actual C++ bindings are located in `*.{hpp, cpp}` files in the root directory
 + `scripts` directory contains per-triplet additional configurations for `make`
 + `codegen` directory is a Rust micro-library that does code generation
-+ `benchmark` directory contains a set of scripts to compare performance of Rust vs C vs Ripper
++ `benchmark` directory contains a set of scripts to compare performance of Rust vs C++ vs Ripper

codegen/build.mk

@@ -21,13 +21,13 @@ nodes.hpp: $(CODEGEN_DEPS)
 	$(DO_CODEGEN)
 CLEAN += nodes.hpp
 
-ruby-parser-c/src/node.rs: $(CODEGEN_DEPS)
+ruby-parser-cpp/src/node.rs: $(CODEGEN_DEPS)
 	$(DO_CODEGEN)
-CLEAN += ruby-parser-c/src/node.rs
+CLEAN += ruby-parser-cpp/src/node.rs
 
-ruby-parser-c/src/message.rs: $(CODEGEN_DEPS)
+ruby-parser-cpp/src/message.rs: $(CODEGEN_DEPS)
 	$(DO_CODEGEN)
-CLEAN += ruby-parser-c/src/message.rs
+CLEAN += ruby-parser-cpp/src/message.rs
 
 # token_id.hpp codegen
 DO_CODEGEN_TOKEN_IDS = cargo run --example build_token_ids --manifest-path codegen/Cargo.toml

codegen/codegen/messages.hpp.liquid

@@ -14,7 +14,7 @@ namespace lib_ruby_parser
     {
     public:
         {%- if message.fields.size == 0 %}
-        // dummy byte (zero-sized structs are no allowed in C)
+        // dummy byte (zero-sized classes are not allowed in C++)
         uint8_t dummy;
         {%- endif %}
 

ruby-parser-cpp/Cargo.toml

@@ -11,7 +11,7 @@ crate-type = ["staticlib"]
 [features]
 default = []
 
-# Include test APIs to run C tests
+# Include test APIs to run C++ tests
 tests = []
 
 [dependencies]

ruby-parser-cpp/build.mk

@@ -2,7 +2,7 @@ RUST_SRCS = $(wildcard ruby-parser-cpp/src/*.rs)
 RUST_SRCS += ruby-parser-cpp/src/node.rs
 RUST_SRCS += ruby-parser-cpp/src/message.rs
 
-ruby-parser-cpp/lib_ruby_parser_cpp.$(A): $(RUST_SRCS)
+ruby-parser-cpp/lib_ruby_parser_cpp.$(A): $(RUST_SRCS) ruby-parser-cpp/src/message.rs ruby-parser-cpp/src/node.rs
 	RUSTFLAGS="$(RUSTFLAGS)" cargo build $(CARGOFLAGS) --manifest-path ruby-parser-cpp/Cargo.toml
 	ls -l ruby-parser-cpp/target/$(TARGET)
 	ls -l ruby-parser-cpp/target/$(TARGET)/$(RUST_ENV)

tests/test.cpp

@@ -27,7 +27,7 @@ extern "C"
 int main()
 {
     lib_ruby_parser__test__print_build_info();
-    std::cerr << "C build info:\n";
+    std::cerr << "C++ build info:\n";
     std::cerr << "sizeof(size_t): " << sizeof(size_t) << "\n";
     std::cerr << "sizeof(void*): " << sizeof(void *) << "\n";
     std::cerr << "\n";