Skip to content

Commit fe37146

Browse files
committed
Allow possibly incorrectly overriding property setters as well
For backwards compatibility.
1 parent ee9b36e commit fe37146

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

crates/header-translator/src/method.rs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -760,8 +760,10 @@ impl Method {
760760
?ty,
761761
"property setter with unsafe arg was marked as safe"
762762
);
763+
true // TODO(breaking): Change to false
764+
} else {
765+
false
763766
}
764-
false
765767
} else if unsafe_retained && !ty.is_primitive_or_record() {
766768
// We could _probably_ mark these as safe, but let's not
767769
// for now, they interact weirdly with the getters (which
@@ -771,8 +773,10 @@ impl Method {
771773
?selector,
772774
"`unsafe_retained` property setter was marked as safe"
773775
);
776+
true // TODO(breaking): Change to false
777+
} else {
778+
false
774779
}
775-
false
776780
} else if let Some(unsafe_) = setter_data.unsafe_ {
777781
// Allow overriding safety with config.
778782
!unsafe_

0 commit comments

Comments
 (0)