)]}'
{
  "commit": "fba1c434a8ab0e0cc67cee08833a0cdc156aab52",
  "tree": "652bab21f5a2dfa83e16be86827d7613925fed3d",
  "parents": [
    "2030ad680997c087c1e15b5f70b7ef01a4b800f4"
  ],
  "author": {
    "name": "Jing Lu",
    "email": "lujin@google.com",
    "time": "Thu Nov 02 16:07:30 2023 -0700"
  },
  "committer": {
    "name": "Copybara-Service",
    "email": "copybara-worker@google.com",
    "time": "Thu Nov 02 16:08:03 2023 -0700"
  },
  "message": "rs_bindings_from_cc: Add `#[allow(unused_imports)]` to generated re-export statements.\n\n[A recent rustc commit](https://github.com/rust-lang/rust/pull/116033/commits/482275b19422b871b986ec0400257a9b80080218) began to flag re-exports of empty modules as a case of unused_imports, which is a warning that cause Crubit-generated code to fail to compile: Crubit may fail to generate bindings for symbols in C++ namespaces but we\u0027d still generate `mod __helper_mod_index` for it and `pub mod canonical_mod`.\n\nAdd `#[allow(unused_import)]` to disable it for now to avoid CI failure. However, in the long term, we should check if module to be reexported is empty (transitively) before emitting the re-export, to make generated bindings more ergonomic and avoid hiding other kinds of unused_imports errors.\n\nPiperOrigin-RevId: 579007904\nChange-Id: I34488cad3a2b68030e5bdf58f9c1543467b26803\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "4a73a3f54c4a679814c42761581888e2053ec604",
      "old_mode": 33188,
      "old_path": "rs_bindings_from_cc/src_code_gen.rs",
      "new_id": "36860beb6089137fd15d3c34fd6c3e8197062d54",
      "new_mode": 33188,
      "new_path": "rs_bindings_from_cc/src_code_gen.rs"
    },
    {
      "type": "modify",
      "old_id": "d0f2427304316b47db43b59128c13ce2a52deafe",
      "old_mode": 33188,
      "old_path": "rs_bindings_from_cc/test/golden/namespace_rs_api.rs",
      "new_id": "0a88799fc2b97540a07c88abaac9845eaf183b8e",
      "new_mode": 33188,
      "new_path": "rs_bindings_from_cc/test/golden/namespace_rs_api.rs"
    }
  ]
}