Skip to content

Commit

Permalink
remove unused methods
Browse files Browse the repository at this point in the history
  • Loading branch information
elfedy committed Jan 29, 2025
1 parent b83538c commit 0562ba6
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 19 deletions.
10 changes: 1 addition & 9 deletions crates/config/src/zksync.rs
Original file line number Diff line number Diff line change
Expand Up @@ -181,14 +181,6 @@ pub fn config_zksolc_settings(config: &Config) -> Result<ZkSolcSettings, SolcErr
config.zksync.settings(libraries, config.evm_version, config.via_ir, config.offline)
}

/// Return the configured `zksolc` compiler
///
/// If not `offline`, will install the default version automatically
/// Will fallback to `zksolc` present in the environment
pub fn config_zksolc_compiler(config: &Config) -> Result<ZkSolcCompiler, SolcError> {
Ok(ZkSolcCompiler { solc: config_solc_compiler(config)? })
}

/// Create a new ZKsync project
pub fn config_create_project(
config: &Config,
Expand Down Expand Up @@ -216,7 +208,7 @@ pub fn config_create_project(
builder = builder.sparse_output(filter);
}

let zksolc_compiler = config_zksolc_compiler(config)?;
let zksolc_compiler = ZkSolcCompiler { solc: config_solc_compiler(config)? };

let project = builder.build(zksolc_compiler)?;

Expand Down
10 changes: 0 additions & 10 deletions crates/zksync/compilers/src/compilers/zksolc/settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -132,16 +132,6 @@ impl ZkSolcSettings {
Ok(Self { settings, cli_settings, zksolc_path, zksolc_version })
}

/// Initialize settings for a given zksolc version
pub fn new_from_version(
settings: ZkSettings,
cli_settings: solc::CliSettings,
zksolc_version: Version,
) -> Result<Self> {
let zksolc_path = ZkSolc::get_path_for_version(&zksolc_version)?;
Ok(Self { settings, cli_settings, zksolc_path, zksolc_version })
}

/// Get zksolc path
pub fn zksolc_path(&self) -> PathBuf {
self.zksolc_path.clone()
Expand Down

0 comments on commit 0562ba6

Please sign in to comment.