Skip to content

Commit 173c600

Browse files
authored
Merge pull request #898 from Universal-Debloater-Alliance/hotfix/conditional-condition-panic
fix: fix all `cfg` shenanigans
2 parents 9fbf8cf + 7506112 commit 173c600

File tree

2 files changed

+44
-35
lines changed

2 files changed

+44
-35
lines changed

src/core/adb.rs

+4-1
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,10 @@ impl ACommand {
150150
// typically 5 allocs.
151151
// ideally 0, if we didn't use `lines`.
152152
.map(|(i, ln)| {
153-
debug_assert!(match i {
153+
// DO NOT "REFACTOR" TO `debug_assert`!
154+
// it's not the same!
155+
#[cfg(debug_assertions)]
156+
assert!(match i {
154157
0 => TRIPLE.is_match(ln),
155158
1 => DISTRO.is_match(ln),
156159
2 =>

src/gui/views/about.rs

+40-34
Original file line numberDiff line numberDiff line change
@@ -46,33 +46,40 @@ impl About {
4646
let uad_lists_btn = button_primary("Update").on_press(Message::UpdateUadLists);
4747

4848
#[cfg(feature = "self-update")]
49-
let self_update_btn = button_primary("Update").on_press(Message::DoSelfUpdate);
50-
51-
#[cfg(feature = "self-update")]
52-
let uad_version_text =
53-
text(format!("{NAME} version: v{}", env!("CARGO_PKG_VERSION"))).width(250);
54-
55-
#[cfg(feature = "self-update")]
56-
#[rustfmt::skip]
57-
let self_update_text = update_state.self_update.latest_release.as_ref().map_or_else(||
58-
if update_state.self_update.status == SelfUpdateStatus::Done {
59-
"(No update available)".to_string()
60-
} else {
61-
update_state.self_update.status.to_string()
62-
}, |r| if update_state.self_update.status == SelfUpdateStatus::Updating {
63-
update_state.self_update.status.to_string()
64-
} else {
65-
format!("({} available)", r.tag_name)
66-
});
67-
68-
#[cfg(feature = "self-update")]
69-
let last_self_update_text = text(self_update_text).style(style::Text::Default);
70-
71-
#[cfg(feature = "self-update")]
72-
let self_update_row = row![uad_version_text, self_update_btn, last_self_update_text,]
73-
.align_items(Alignment::Center)
74-
.spacing(10)
75-
.width(550);
49+
let self_update_row = {
50+
let self_update_btn = button_primary("Update").on_press(Message::DoSelfUpdate);
51+
52+
let uad_version_text =
53+
text(format!("{NAME} version: v{}", env!("CARGO_PKG_VERSION"))).width(250);
54+
55+
let self_update_text = update_state
56+
.self_update
57+
.latest_release
58+
.as_ref()
59+
.map_or_else(
60+
|| {
61+
if update_state.self_update.status == SelfUpdateStatus::Done {
62+
"(No update available)".to_string()
63+
} else {
64+
update_state.self_update.status.to_string()
65+
}
66+
},
67+
|r| {
68+
if update_state.self_update.status == SelfUpdateStatus::Updating {
69+
update_state.self_update.status.to_string()
70+
} else {
71+
format!("({} available)", r.tag_name)
72+
}
73+
},
74+
);
75+
76+
let last_self_update_text = text(self_update_text).style(style::Text::Default);
77+
78+
row![uad_version_text, self_update_btn, last_self_update_text,]
79+
.align_items(Alignment::Center)
80+
.spacing(10)
81+
.width(550)
82+
};
7683

7784
let uad_list_row = row![uad_list_text, uad_lists_btn, last_update_text,]
7885
.align_items(Alignment::Center)
@@ -105,13 +112,12 @@ impl About {
105112
.align_items(Alignment::Center)
106113
.width(550);
107114

108-
let update_column = if cfg!(feature = "self-update") {
109-
column![uad_list_row, self_update_row, adb_version_row]
110-
} else {
111-
column![uad_list_row, adb_version_row]
112-
}
113-
.align_items(Alignment::Center)
114-
.spacing(10);
115+
#[cfg(feature = "self-update")]
116+
let update_column = column![uad_list_row, self_update_row, adb_version_row];
117+
#[cfg(not(feature = "self-update"))]
118+
let update_column = column![uad_list_row, adb_version_row];
119+
120+
let update_column = update_column.align_items(Alignment::Center).spacing(10);
115121

116122
let update_container = container(update_column)
117123
.width(Length::Fill)

0 commit comments

Comments
 (0)