Skip to content

Commit

Permalink
Python: Use __repr__ instead of __str__ in classes. (#2)
Browse files Browse the repository at this point in the history
  • Loading branch information
overcat authored May 13, 2024
1 parent c95ec1f commit c5f88c2
Show file tree
Hide file tree
Showing 46 changed files with 120 additions and 120 deletions.
6 changes: 3 additions & 3 deletions lib/xdrgen/generators/python.rb
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ def __eq__(self, other: object):
return NotImplemented
return self.#{typedef_name_underscore} == other.#{typedef_name_underscore}
def __str__(self):
def __repr__(self):
return f"<#{typedef_name} [#{typedef_name_underscore}={self.#{typedef_name_underscore}}]>"
HEREDOC
end
Expand Down Expand Up @@ -296,7 +296,7 @@ def __eq__(self, other: object):
return #{attribute_names.map { |m| 'self.' + m + '== other.' + m }.join(" and ")}
HEREDOC

out.puts "def __str__(self):"
out.puts "def __repr__(self):"
out.indent(2) do
out.puts "out = []"
out.puts "out.append(f'#{union_discriminant_name_underscore}={self.#{union_discriminant_name_underscore}}')"
Expand Down Expand Up @@ -389,7 +389,7 @@ def __eq__(self, other: object):
return #{attribute_names.map { |m| 'self.' + m + '== other.' + m }.join(" and ")}
HEREDOC

out.puts "def __str__(self):"
out.puts "def __repr__(self):"
out.indent(2) do
out.puts "out = ["
out.indent(2) do
Expand Down
18 changes: 9 additions & 9 deletions lib/xdrgen/generators/python/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Integer [value={self.value}]>"


Expand All @@ -55,7 +55,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<UnsignedInteger [value={self.value}]>"


Expand All @@ -78,7 +78,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Float [value={self.value}]>"


Expand All @@ -101,7 +101,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Double [value={self.value}]>"


Expand All @@ -124,7 +124,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Hyper [value={self.value}]>"


Expand All @@ -147,7 +147,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<UnsignedHyper [value={self.value}]>"


Expand All @@ -170,7 +170,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Boolean [value={self.value}]>"


Expand Down Expand Up @@ -201,7 +201,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value and self.size == other.size

def __str__(self):
def __repr__(self):
return f"<String [value={self.value}, size={self.size}]>"


Expand Down Expand Up @@ -248,5 +248,5 @@ def __eq__(self, other: object) -> bool:
and self.size == other.size
)

def __str__(self):
def __repr__(self):
return f"<Opaque [value={self.value}, fixed={self.fixed}, size={self.size}]>"
18 changes: 9 additions & 9 deletions spec/output/generator_spec_python/block_comments.x/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Integer [value={self.value}]>"


Expand All @@ -55,7 +55,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<UnsignedInteger [value={self.value}]>"


Expand All @@ -78,7 +78,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Float [value={self.value}]>"


Expand All @@ -101,7 +101,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Double [value={self.value}]>"


Expand All @@ -124,7 +124,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Hyper [value={self.value}]>"


Expand All @@ -147,7 +147,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<UnsignedHyper [value={self.value}]>"


Expand All @@ -170,7 +170,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Boolean [value={self.value}]>"


Expand Down Expand Up @@ -201,7 +201,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value and self.size == other.size

def __str__(self):
def __repr__(self):
return f"<String [value={self.value}, size={self.size}]>"


Expand Down Expand Up @@ -248,5 +248,5 @@ def __eq__(self, other: object) -> bool:
and self.size == other.size
)

def __str__(self):
def __repr__(self):
return f"<Opaque [value={self.value}, fixed={self.fixed}, size={self.size}]>"
18 changes: 9 additions & 9 deletions spec/output/generator_spec_python/const.x/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Integer [value={self.value}]>"


Expand All @@ -55,7 +55,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<UnsignedInteger [value={self.value}]>"


Expand All @@ -78,7 +78,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Float [value={self.value}]>"


Expand All @@ -101,7 +101,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Double [value={self.value}]>"


Expand All @@ -124,7 +124,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Hyper [value={self.value}]>"


Expand All @@ -147,7 +147,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<UnsignedHyper [value={self.value}]>"


Expand All @@ -170,7 +170,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Boolean [value={self.value}]>"


Expand Down Expand Up @@ -201,7 +201,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value and self.size == other.size

def __str__(self):
def __repr__(self):
return f"<String [value={self.value}, size={self.size}]>"


Expand Down Expand Up @@ -248,5 +248,5 @@ def __eq__(self, other: object) -> bool:
and self.size == other.size
)

def __str__(self):
def __repr__(self):
return f"<Opaque [value={self.value}, fixed={self.fixed}, size={self.size}]>"
2 changes: 1 addition & 1 deletion spec/output/generator_spec_python/const.x/test_array.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,5 +56,5 @@ def __eq__(self, other: object):
return NotImplemented
return self.test_array == other.test_array

def __str__(self):
def __repr__(self):
return f"<TestArray [test_array={self.test_array}]>"
2 changes: 1 addition & 1 deletion spec/output/generator_spec_python/const.x/test_array2.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,5 +57,5 @@ def __eq__(self, other: object):
return NotImplemented
return self.test_array2 == other.test_array2

def __str__(self):
def __repr__(self):
return f"<TestArray2 [test_array2={self.test_array2}]>"
18 changes: 9 additions & 9 deletions spec/output/generator_spec_python/enum.x/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Integer [value={self.value}]>"


Expand All @@ -55,7 +55,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<UnsignedInteger [value={self.value}]>"


Expand All @@ -78,7 +78,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Float [value={self.value}]>"


Expand All @@ -101,7 +101,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Double [value={self.value}]>"


Expand All @@ -124,7 +124,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Hyper [value={self.value}]>"


Expand All @@ -147,7 +147,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<UnsignedHyper [value={self.value}]>"


Expand All @@ -170,7 +170,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value

def __str__(self):
def __repr__(self):
return f"<Boolean [value={self.value}]>"


Expand Down Expand Up @@ -201,7 +201,7 @@ def __eq__(self, other: object) -> bool:
return NotImplemented
return self.value == other.value and self.size == other.size

def __str__(self):
def __repr__(self):
return f"<String [value={self.value}, size={self.size}]>"


Expand Down Expand Up @@ -248,5 +248,5 @@ def __eq__(self, other: object) -> bool:
and self.size == other.size
)

def __str__(self):
def __repr__(self):
return f"<Opaque [value={self.value}, fixed={self.fixed}, size={self.size}]>"
Loading

0 comments on commit c5f88c2

Please sign in to comment.