Skip to content

1. Allow absolute paths; 2. File not closed; 3. ANSI escape sequences not stripped #2

@paul-j-lucas

Description

@paul-j-lucas

There are a few problems with this code:

  1. It should allow absolute paths. Specifically, it should not prepend ./ to the path. It should simply leave the path alone.
  2. The file isn't closed.
  3. When you send the output to a file, LLDB includes what it would have sent to the terminal verbatim. For a color terminal, this includes ANSI escape sequences. These should be stripped from the output.

I would have created a branch and submitted a pull request, but don't have write-permission to the repo; so here's a patch.
patch.txt

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions