You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We currently pretty print fixpoint horn constraints in the Display implementation, i.e., we add newlines and indentation to make the sexpr easier to read. This is wasteful if we are not looking at the constraint and it has caused problems in the past (e.g., #857 (comment)). I still think is valuable to pretty print them in cases where we do look at them. We should disable pretty printing by default and implement a flag to enable it on demand.
Additionally, the PadAdapter is probably not the most efficient implementation. We could use normal functions instead of implementing Display to avoid wrapping the formatter to get the padding behavior.
The text was updated successfully, but these errors were encountered:
We currently pretty print fixpoint horn constraints in the
Display
implementation, i.e., we add newlines and indentation to make the sexpr easier to read. This is wasteful if we are not looking at the constraint and it has caused problems in the past (e.g., #857 (comment)). I still think is valuable to pretty print them in cases where we do look at them. We should disable pretty printing by default and implement a flag to enable it on demand.Additionally, the
PadAdapter
is probably not the most efficient implementation. We could use normal functions instead of implementingDisplay
to avoid wrapping the formatter to get the padding behavior.The text was updated successfully, but these errors were encountered: