diff options
| author | Michael Krasnitski <michael.krasnitski@gmail.com> | 2023-01-06 14:13:55 -0500 |
|---|---|---|
| committer | KyleMiles <krm504@nyu.edu> | 2023-01-06 18:04:05 -0500 |
| commit | ad7f5293db22e23d8d50077aa99f5d7a904092fc (patch) | |
| tree | 294217bc06b0cdc3a86ebdb1132ad8cddd02d5ce /rust/src/downloadprovider.rs | |
| parent | c97b7d878c048312257843ee585280cc07a906e7 (diff) | |
Rust API: Additional Cleanup
Diffstat (limited to 'rust/src/downloadprovider.rs')
| -rw-r--r-- | rust/src/downloadprovider.rs | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/rust/src/downloadprovider.rs b/rust/src/downloadprovider.rs index dfc8b5e2..7de8704c 100644 --- a/rust/src/downloadprovider.rs +++ b/rust/src/downloadprovider.rs @@ -38,9 +38,8 @@ impl DownloadProvider { Ok(unsafe { Array::new(list, count, ()) }) } - /// TODO : Clippy isn't happy...says we should `impl Default`....excessive error checking might be preventing us from doing so - #[allow(clippy::should_implement_trait)] - pub fn default() -> Result<DownloadProvider, ()> { + /// TODO : We may want to `impl Default`....excessive error checking might be preventing us from doing so + pub fn try_default() -> Result<DownloadProvider, ()> { let s = Settings::new(""); let dp_name = s.get_string("network.downloadProviderName", None, None); Self::get(dp_name).ok_or(()) |