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

Remove comments from mock pass #1030

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Remove comments from mock pass #1030

wants to merge 2 commits into from

Commits on Dec 4, 2019

  1. Add RemoveCommentsFromMockPass 

    Add a new string manipulation pass that removes all code comments and
empty lines from the generated mock code, so that the other string
manipulation passes have less code to work on.
    @robertbasic
    robertbasic committed Dec 4, 2019
    Configuration menu
    Copy the full SHA
    98e74e6 View commit details
    Browse the repository at this point in the history

  2. Split the code by the first curly bracket 

    Given that the str_replace has to happen on substrings that are before
the first curly bracket, split the generated mock code by the first
curly bracket, and then do the str_replace only on that smaller chunk of
code.

Not sure if there's really any benefit to this, as we're replacing one
string operation with another one.
    @robertbasic
    robertbasic committed Dec 4, 2019
    Configuration menu
    Copy the full SHA
    939187a View commit details
    Browse the repository at this point in the history