Skip to content

Commit 9fd31b8

Browse files
authored
Merge pull request RustPython#3911 from moreal/use-getset-instead-property-word
Unify elements referring to getset to `GetSet`
2 parents 242e511 + 5d2146e commit 9fd31b8

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

derive/src/pyclass.rs

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,7 @@ struct MethodItem {
404404
}
405405

406406
/// #[pyproperty]
407-
struct PropertyItem {
407+
struct GetSetItem {
408408
inner: ContentItemInner<AttrName>,
409409
}
410410

@@ -429,7 +429,7 @@ impl ContentItem for MethodItem {
429429
&self.inner
430430
}
431431
}
432-
impl ContentItem for PropertyItem {
432+
impl ContentItem for GetSetItem {
433433
type AttrName = AttrName;
434434
fn inner(&self) -> &ContentItemInner<AttrName> {
435435
&self.inner
@@ -521,7 +521,7 @@ where
521521
}
522522
}
523523

524-
impl<Item> ImplItem<Item> for PropertyItem
524+
impl<Item> ImplItem<Item> for GetSetItem
525525
where
526526
Item: ItemLike + ToTokens + GetIdent,
527527
{
@@ -533,9 +533,9 @@ where
533533
let ident = &func.sig().ident;
534534

535535
let item_attr = args.attrs.remove(self.index());
536-
let item_meta = PropertyItemMeta::from_attr(ident.clone(), &item_attr)?;
536+
let item_meta = GetSetItemMeta::from_attr(ident.clone(), &item_attr)?;
537537

538-
let (py_name, kind) = item_meta.property_name()?;
538+
let (py_name, kind) = item_meta.getset_name()?;
539539
args.context
540540
.getset_items
541541
.add_item(py_name, args.cfgs.to_vec(), kind, ident.clone())?;
@@ -706,7 +706,7 @@ impl GetSetNursery {
706706
if getter.is_none() {
707707
errors.push(syn::Error::new_spanned(
708708
setter.as_ref().or(deleter.as_ref()).unwrap(),
709-
format!("Property '{}' is missing a getter", name),
709+
format!("GetSet '{}' is missing a getter", name),
710710
));
711711
};
712712
}
@@ -781,9 +781,9 @@ impl MethodItemMeta {
781781
}
782782
}
783783

784-
struct PropertyItemMeta(ItemMetaInner);
784+
struct GetSetItemMeta(ItemMetaInner);
785785

786-
impl ItemMeta for PropertyItemMeta {
786+
impl ItemMeta for GetSetItemMeta {
787787
const ALLOWED_NAMES: &'static [&'static str] = &["name", "magic", "setter", "deleter"];
788788

789789
fn from_inner(inner: ItemMetaInner) -> Self {
@@ -794,8 +794,8 @@ impl ItemMeta for PropertyItemMeta {
794794
}
795795
}
796796

797-
impl PropertyItemMeta {
798-
fn property_name(&self) -> Result<(String, GetSetItemKind)> {
797+
impl GetSetItemMeta {
798+
fn getset_name(&self) -> Result<(String, GetSetItemKind)> {
799799
let inner = self.inner();
800800
let magic = inner._bool("magic")?;
801801
let kind = match (inner._bool("setter")?, inner._bool("deleter")?) {
@@ -1032,7 +1032,7 @@ where
10321032
inner: ContentItemInner { index, attr_name },
10331033
})
10341034
}
1035-
GetSet => Box::new(PropertyItem {
1035+
GetSet => Box::new(GetSetItem {
10361036
inner: ContentItemInner { index, attr_name },
10371037
}),
10381038
Slot => Box::new(SlotItem {

0 commit comments

Comments
 (0)