transpile: don't hide --no-reorganize-definitions and rename _{no => unused}_reorganize_definitions

The latter is because the two fields are equal,
so naming the second one `no` is misleading.

Author: kkysen

c2rust/src/bin/c2rust-transpile.rs

@@ -1,4 +1,4 @@
-use clap::{Parser, ValueEnum};
+use clap::{ArgAction, Parser, ValueEnum};
 use log::LevelFilter;
 use regex::Regex;
 use std::{ffi::OsStr, fs, path::PathBuf};
@@ -125,7 +125,7 @@ struct Args {
     reduce_type_annotations: bool,
 
     /// Disable `--reorganize-definitions`
-    #[clap(long = "no-reorganize-definitions", hidden = true, action = clap::ArgAction::SetFalse)]
+    #[clap(long = "no-reorganize-definitions", action = ArgAction::SetFalse)]
     reorganize_definitions: bool, // NB, this *is* the right field; this flag sets it to *false*.
 
     /// Output file in such a way that the refactoring tool can deduplicate code (enabled by default; disable with `--no-reorganize-definitions`)
@@ -134,7 +134,7 @@ struct Args {
         long = "reorganize-definitions",
         conflicts_with = "reorganize-definitions"
     )]
-    _no_reorganize_definitions: bool, // Field is unused, but flag mutually excludes the negative sense above.
+    _unused_reorganize_definitions: bool, // Field is unused, but flag mutually excludes the negative sense above.
 
     /// Extra arguments to pass to clang frontend during parsing the input C file
     #[clap(multiple = true, last(true))]