Skip to content

Redo some string operations #439

New issue
New issue
Closed
Closed
Redo some string operations#439
Labels
bitesizeprio:low

Description

@sebastinas
sebastinas
opened on Jan 11, 2015

In bpython/curtsiesfrontend/replyplainter.py, some string operations could be simplified by using ljust or format.

In

bpython/bpython/curtsiesfrontend/replpainter.py

Line 159 in 566dfe5

output_lines.append(border_color(config.left_border+u' ')+((line+' '*(width - len(line)))[:width])+border_color(u' ' + config.right_border))
line+' '*(width - len(line)))[:width]) can be replaced by line.ljust(width)[:width] (untested).

Activity

  1. sebastinas
    added
    prio:low
    bitesize
    blocked
    on Jan 11, 2015

  2. sebastinas commented on Jan 15, 2015

    @sebastinas
    sebastinas
    on Jan 15, 2015
    ContributorAuthor

    Blocked by https://github.com/thomasballinger/curtsies/issues/62.

  3. thomasballinger commented on Jan 20, 2015

    @thomasballinger
    thomasballinger
    on Jan 20, 2015
    Member

    No longer blocked, Curtsies 0.1.17 adds ljust and rjust that work as described here. (but bpython not updated yet)

  4. thomasballinger
    removed
    blocked
    on Jan 20, 2015
  5. sebastinas
    closed this as completedin 8b5df4fon Jan 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bitesizeprio:low

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions