Resolve FIXMEs

core/lib/contract_verifier/src/compilers/mod.rs

@@ -77,7 +77,6 @@ fn parse_standard_json_output(
             .iter()
             .any(|err| err["severity"].as_str() == Some("error"))
         {
-            // FIXME: collected messages contain warnings as well; is this intentional?
             let error_messages = errors
                 .into_iter()
                 .filter_map(|err| {

core/lib/contract_verifier/src/compilers/vyper.rs

@@ -31,7 +31,6 @@ impl VyperInput {
             file_name,
             sources,
             optimizer_mode: if req.optimization_used {
-                // FIXME: values have differing semantics compared to other compilers; is this OK?
                 req.optimizer_mode
             } else {
                 // `none` mode is not the default mode (which is `gas`), so we must specify it explicitly here

core/lib/types/src/contract_verification_api.rs

@@ -137,6 +137,8 @@ pub struct VerificationIncomingRequest {
     #[serde(flatten)]
     pub compiler_versions: CompilerVersions,
     pub optimization_used: bool,
+    /// Optimization mode used for the contract. Semantics depends on the compiler used; e.g., for `vyper`,
+    /// allowed values are `gas` (default), `codesize` or `none`.
     pub optimizer_mode: Option<String>,
     #[serde(default)]
     pub constructor_arguments: Bytes,