diff --git a/pymbolic/mapper/c_code.py b/pymbolic/mapper/c_code.py index 4a1b9b7ec22c764f045add605344f9f30fd2a99a..bae2d108eecbd205444c36f5b38710db6d46a6ff 100644 --- a/pymbolic/mapper/c_code.py +++ b/pymbolic/mapper/c_code.py @@ -67,12 +67,11 @@ class CCodeMapper(SimplifyingSortingStringifyMapper): # Let's see how bad of an idea this is--sane people would only # apply this to integers, right? - from pymbolic.mapper.stringifier import PREC_PRODUCT, PREC_POWER - return self.parenthesize_if_needed( - self.format("%s/%s", - self.rec(expr.numerator, PREC_PRODUCT), - self.rec(expr.denominator, PREC_POWER)), # analogous to ^{-1} - enclosing_prec, PREC_PRODUCT) + from pymbolic.mapper.stringifier import ( + PREC_PRODUCT, PREC_POWER) + return self.format("(%s/%s)", + self.rec(expr.numerator, PREC_PRODUCT), + self.rec(expr.denominator, PREC_POWER)) # analogous to ^{-1} def map_common_subexpression(self, expr, enclosing_prec): try: diff --git a/pymbolic/mapper/stringifier.py b/pymbolic/mapper/stringifier.py index 677d8b4aab7d53a2ac6199a2186a25f1341b7d5c..b6c5c0bbd30e2793d5eb613e4b9d130f6beca2fe 100644 --- a/pymbolic/mapper/stringifier.py +++ b/pymbolic/mapper/stringifier.py @@ -112,11 +112,9 @@ class StringifyMapper(pymbolic.mapper.RecursiveMapper): enclosing_prec, PREC_POWER) def map_remainder(self, expr, enclosing_prec): - return self.parenthesize_if_needed( - self.format("%s %% %s", + return self.format("(%s %% %s)", self.rec(expr.numerator, PREC_PRODUCT), - self.rec(expr.denominator, PREC_POWER)), # analogous to ^{-1} - enclosing_prec, PREC_PRODUCT) + self.rec(expr.denominator, PREC_POWER)) # analogous to ^{-1} def map_polynomial(self, expr, enclosing_prec): from pymbolic.primitives import flattened_sum