Delete the file.rs comment and single_file flag

Author: lucic71

cpp2rust/ast_consumer.cpp

@@ -7,8 +7,7 @@
 
 namespace cpp2rust {
 void ASTConsumer::HandleTranslationUnit(clang::ASTContext &ctx) {
-  auto converter =
-      CreateConverter(rs_code_, ctx, model_, single_file_, rules_dir_);
+  auto converter = CreateConverter(rs_code_, ctx, model_, rules_dir_);
   converter->SetSema(CI_.getSema());
   if (first_) {
     converter->EmitFilePreamble();

cpp2rust/ast_consumer.h

@@ -15,18 +15,17 @@ namespace cpp2rust {
 class ASTConsumer : public clang::ASTConsumer {
 public:
   explicit ASTConsumer(std::string &rs_code, Model model, bool first,
-                       bool single_file, clang::CompilerInstance &CI,
+                       clang::CompilerInstance &CI,
                        const std::string &rules_dir)
       : rs_code_(rs_code), model_(model), first_(first),
-        single_file_(single_file), CI_(CI), rules_dir_(rules_dir) {}
+        CI_(CI), rules_dir_(rules_dir) {}
 
   void HandleTranslationUnit(clang::ASTContext &ctx) override;
 
 private:
   std::string &rs_code_;
   Model model_;
   bool first_;
-  bool single_file_;
   clang::CompilerInstance &CI_;
   const std::string &rules_dir_;
 };

cpp2rust/converter/converter.cpp

@@ -275,9 +275,6 @@ bool Converter::VisitUsingType(clang::UsingType *type) {
 bool Converter::Convert(clang::Decl *decl) { return TraverseDecl(decl); }
 
 bool Converter::VisitTranslationUnitDecl(clang::TranslationUnitDecl *decl) {
-  if (!single_file_) {
-    StrCat("\n//", GetMainFileName(ctx_) + ".rs\n");
-  }
   for (auto *child : decl->decls()) {
     if (IsConvertibleDecl(child) &&
         (IsInMainFile(child) || !decl_ids_.contains(GetID(child)))) {

cpp2rust/converter/converter.h

@@ -24,12 +24,11 @@ class Converter : public clang::RecursiveASTVisitor<Converter> {
 
 public:
   explicit Converter(std::string &rs_code, clang::ASTContext &ctx,
-                     bool single_file = false,
                      const char *keyword_unsafe = "unsafe",
                      const char *keyword_mut = "mut",
                      const char *keyword_ptr_decay = ".as_mut_ptr()",
                      const char *keyword_const_fn = keyword::kConst)
-      : rs_code_(&rs_code), ctx_(ctx), single_file_(single_file),
+      : rs_code_(&rs_code), ctx_(ctx),
         keyword_unsafe_(keyword_unsafe), keyword_mut_(keyword_mut),
         keyword_ptr_decay_(keyword_ptr_decay),
         keyword_const_fn_(keyword_const_fn) {}
@@ -527,7 +526,6 @@ class Converter : public clang::RecursiveASTVisitor<Converter> {
 
   std::string *rs_code_;
   clang::ASTContext &ctx_;
-  bool single_file_ = false;
   clang::FunctionDecl *curr_function_ = nullptr;
   bool in_function_formals_ = false;
   bool in_const_initializer_ = false;

cpp2rust/converter/factory.cpp

@@ -10,14 +10,13 @@ namespace cpp2rust {
 
 std::unique_ptr<Converter> CreateConverter(std::string &rs_code,
                                            clang::ASTContext &ctx, Model model,
-                                           bool single_file,
                                            const std::string &rules_dir) {
   Mapper::LoadTranslationRules(model, ctx, rules_dir);
   switch (model) {
   case Model::kUnsafe:
-    return std::make_unique<Converter>(rs_code, ctx, single_file);
+    return std::make_unique<Converter>(rs_code, ctx);
   case Model::kRefCount:
-    return std::make_unique<ConverterRefCount>(rs_code, ctx, single_file);
+    return std::make_unique<ConverterRefCount>(rs_code, ctx);
   }
   std::unreachable();
 }

cpp2rust/converter/factory.h

@@ -18,6 +18,5 @@ class Converter;
 
 std::unique_ptr<Converter> CreateConverter(std::string &rs_code,
                                            clang::ASTContext &ctx, Model model,
-                                           bool single_file,
                                            const std::string &rules_dir);
 } // namespace cpp2rust

cpp2rust/converter/models/converter_refcount.cpp

@@ -14,8 +14,8 @@
 
 namespace cpp2rust {
 ConverterRefCount::ConverterRefCount(std::string &rs_code,
-                                     clang::ASTContext &ctx, bool single_file)
-    : Converter(rs_code, ctx, single_file, "", "", ".as_pointer()", ""),
+                                     clang::ASTContext &ctx)
+    : Converter(rs_code, ctx, "", "", ".as_pointer()", ""),
       conversion_kind_({ConversionKind::Unboxed}) {}
 
 void ConverterRefCount::EmitFilePreamble() {

cpp2rust/converter/models/converter_refcount.h

@@ -8,8 +8,7 @@
 namespace cpp2rust {
 class ConverterRefCount final : public Converter {
 public:
-  ConverterRefCount(std::string &rs_code, clang::ASTContext &ctx,
-                    bool single_file = false);
+  ConverterRefCount(std::string &rs_code, clang::ASTContext &ctx);
 
   void EmitFilePreamble() override;
 

cpp2rust/cpp2rust_lib.cpp

@@ -26,7 +26,7 @@ std::string TranspileSrc(std::string_view cc_code, Model model,
   std::string rs_code;
   clang::tooling::runToolOnCodeWithArgs(
       std::make_unique<FrontendAction>(rs_code, model, /*first=*/true,
-                                       /*single_file=*/true, rules_dir),
+                                       rules_dir),
       cc_code, tool_args, std::filesystem::path(filename).filename().string(),
       filename.ends_with(".c") ? CLANG_C_COMPILER : CLANG_CXX_COMPILER);
   return rs_code;

cpp2rust/frontend_action.h

@@ -18,22 +18,21 @@ namespace cpp2rust {
 class FrontendAction : public clang::ASTFrontendAction {
 public:
   explicit FrontendAction(std::string &rs_code, Model model, bool first,
-                          bool single_file, const std::string &rules_dir)
+                          const std::string &rules_dir)
       : rs_code_(rs_code), model_(model), first_(first),
-        single_file_(single_file), rules_dir_(rules_dir) {}
+        rules_dir_(rules_dir) {}
 
   std::unique_ptr<clang::ASTConsumer>
   CreateASTConsumer(clang::CompilerInstance &CI,
                     llvm::StringRef InFile) override {
-    return std::make_unique<ASTConsumer>(rs_code_, model_, first_, single_file_,
-                                         CI, rules_dir_);
+    return std::make_unique<ASTConsumer>(rs_code_, model_, first_, CI,
+                                         rules_dir_);
   }
 
 private:
   std::string &rs_code_;
   Model model_;
   bool first_;
-  bool single_file_;
   const std::string &rules_dir_;
 };
 
@@ -46,8 +45,7 @@ class FrontendActionFactory : public clang::tooling::FrontendActionFactory {
   std::unique_ptr<clang::FrontendAction> create() override {
     bool f = first_;
     first_ = false;
-    return std::make_unique<FrontendAction>(rs_code_, model_, f,
-                                            /*single_file=*/false, rules_dir_);
+    return std::make_unique<FrontendAction>(rs_code_, model_, f, rules_dir_);
   }
 
 private: