Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix codegen line break #292

Merged
merged 2 commits into from
Oct 21, 2015
Merged

Fix codegen line break #292

merged 2 commits into from
Oct 21, 2015

Commits on Oct 21, 2015

  1. Add test to reproduce linewrap error

    @oliverlee
    oliverlee committed Oct 21, 2015
    Configuration menu
    Copy the full SHA
    12ef6f8 View commit details
    Browse the repository at this point in the history

  2. Add whitespace to generated code to allow line breaks 

    Some generated expressions, particularly those that contain only '*' and
'/' operands, have no whitespace. In this case, the textwrap module may
break the line in the middle of a symbol or mathematical expression if
the line is long, resulting in code that cannot compile. This commit
adds whitespace before and after the '*', '/' operands so that textwrap
will have more options for inserting line breaks. The textwrap option
'break_long_words' is also set to True, allowing long lines in the event
that no whitespace is present.

This commit resolves pydy#263.
    @oliverlee
    oliverlee committed Oct 21, 2015
    Configuration menu
    Copy the full SHA
    e11f2fe View commit details
    Browse the repository at this point in the history