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.

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

Rename TraceChecker to TraceCheck #229

Closed
Heizmann opened this issue Sep 25, 2017 · 9 comments
Closed

Rename TraceChecker to TraceCheck #229

Heizmann opened this issue Sep 25, 2017 · 9 comments
Assignees

Comments

@Heizmann
Copy link
Member

The name TraceChecker is misleading. The name suggests that such an object (once constructed) can be used to check traces.
However, if the construction of the object is finished the object "only" represents the trace check and its public methods only allow you to get input, output and statistical information.

If no one objects I will to the renaming indicated by the title.

@danieldietsch
Copy link
Member

I like it.

@greitsch
Copy link
Member

No objection from me.

@alexandernutz
Copy link
Member

dito

@Heizmann
Copy link
Member Author

Heizmann commented Oct 3, 2017

But I have not done this yet!?!

@danieldietsch
Copy link
Member

But I did it with ee68654.

@Heizmann
Copy link
Member Author

Heizmann commented Oct 3, 2017

Thanks. But one should not only relabel the class name but also all variable names! I can do this if you think it is a waste of time.

@Heizmann Heizmann reopened this Oct 3, 2017
@danieldietsch
Copy link
Member

Oh, you are right. I forgot this, but can do it later.

@danieldietsch
Copy link
Member

Should be finished with d92ef35 -- close if this is what you thought.

@Heizmann
Copy link
Member Author

Heizmann commented Oct 3, 2017

Thanks. I checked with
grep -ir cechecker trunk/source/*/src
and did not find anything.

@Heizmann Heizmann closed this as completed Oct 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants