Ticket #53 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

Allow for immutable subclasses of YAMLObject

Reported by: toidinamai Assigned to: xi
Priority: normal Component: pyyaml
Severity: trivial Keywords:
Cc:

Description

YAMLObject are not directly instantiated so they don't need their own instance attribute (i.e. an own dict). Because of that they should define an empty slots and let subclasses decide whether they want arbitrary instance attributes or not: 

--- lib/yaml/__init__.py        (revision 251)
+++ lib/yaml/__init__.py        (working copy)
@@ -272,6 +272,9 @@
     yaml_tag = None
     yaml_flow_style = None
 
+    # no direct instantiation, so allow for immutable subclasses
+    __slots__ = ()
+
     def from_yaml(cls, loader, node):
         """
         Convert a representation node to a Python object.

The same is basically also true for YAMLObjectMetaclass but I doubt there is much use for this in practice.

Attachments

Change History

05/08/07 16:39:39 changed by xi

  • status changed from new to closed.
  • resolution set to fixed.

Makes sense to me. Applied in [252]. Thanks for the patch.

Add/Change #53 (Allow for immutable subclasses of YAMLObject)




Change Properties
Action