Skip to content

Commit 6c20a76

Browse files
committed
Move an if from the printer
This if is already present in `Mv_latex_str`, and it makes more sense to push it into `Mv_str` to match.
1 parent 6c38568 commit 6c20a76

File tree

2 files changed

+7
-9
lines changed

2 files changed

+7
-9
lines changed

galgebra/mv.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -585,6 +585,10 @@ def __getitem__(self, key):
585585

586586
def Mv_str(self, print_obj):
587587
global print_replace_old, print_replace_new
588+
589+
if self.obj == S.Zero:
590+
return ZERO_STR
591+
588592
if self.i_grade == 0:
589593
return print_obj.doprint(self.obj)
590594

@@ -653,7 +657,7 @@ def Mv_str(self, print_obj):
653657

654658
def Mv_latex_str(self, print_obj):
655659

656-
if self.obj == 0:
660+
if self.obj == S.Zero:
657661
return ZERO_STR
658662

659663
first_line = True

galgebra/printer.py

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -371,10 +371,7 @@ def _print_Matrix(self, expr):
371371
return out_str
372372

373373
def _print_Mv(self, expr):
374-
if expr.obj == S(0):
375-
return ZERO_STR
376-
else:
377-
return expr.Mv_str(self)
374+
return expr.Mv_str(self)
378375

379376
def _print_Pdop(self, expr):
380377
return expr.Pdop_str(self)
@@ -937,10 +934,7 @@ def _print_Derivative(self, expr):
937934
return s
938935

939936
def _print_Mv(self, expr):
940-
if expr.obj == S(0):
941-
return ZERO_STR
942-
else:
943-
return expr.Mv_latex_str(self)
937+
return expr.Mv_latex_str(self)
944938

945939
def _print_Pdop(self, expr):
946940
return expr.Pdop_latex_str(self)

0 commit comments

Comments
 (0)