Fix handling of `std::cmp::Ordering` and `Option<i8>` in `format_ty...`.
Before this CL the "Cross-crate dependencies are not supported yet
(b/258261328)" error had no test coverage. Adding this test coverage is
the primary motivation for this CL. This CL is triggered by the
realization that cross-crate test coverage can be provided by testing
types from the `std` crate (earlier I thought that testing these
scenarios will require modifying `test::run_compiler` to support
depending on other crates).
Adding tests for `std::cmp::Ordering` helped flush out an additional
bug: `format_ty_for_rs` incorrectly assumed that `ty.to_string()`
returns a single identifier and panicked when it returned
"std::cmp::Ordering". Fixing this bug encouraged introduction of
initial, minimal support for modules/namespaces and reusing
`NamespaceQualifier` from `rs_bindings_from_cc`.
The CL also opportunistucally adds test coverage for _generic_
types from other crates - by testing `Option<i8>`.
PiperOrigin-RevId: 491660080
diff --git a/cc_bindings_from_rs/cc_bindings_from_rs.rs b/cc_bindings_from_rs/cc_bindings_from_rs.rs
index 9fc4a88..b878a34 100644
--- a/cc_bindings_from_rs/cc_bindings_from_rs.rs
+++ b/cc_bindings_from_rs/cc_bindings_from_rs.rs
@@ -381,7 +381,7 @@
#[no_mangle]
extern "C" fn __crubit_thunk__RNvCslKXfKXPWofF_10test_crate15public_function() -> () {
- test_crate::public_function()
+ ::test_crate::public_function()
}
"#
);
