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

Comments and refactorings (hidden -> masked) in the TextInput. #82

Closed
wants to merge 3 commits into from

Conversation

perovic
Copy link
Contributor

@perovic perovic commented Sep 1, 2015

Passes cling test and roottest.

@Axel-Naumann
Copy link
Member

Thanks!

Do we also need changes in the Windows part?

@perovic
Copy link
Contributor Author

perovic commented Sep 1, 2015

Not for now, code changes were only made in the platform-agnostic part.

@Axel-Naumann
Copy link
Member

Awesome, thanks SO much!

Merged (after squash)!

@perovic
Copy link
Contributor Author

perovic commented Sep 1, 2015

My pleasure 👍
On to the next one!

@perovic perovic deleted the TextInputComments branch September 1, 2015 12:56
Dr15Jones pushed a commit to Dr15Jones/root that referenced this pull request Sep 22, 2015
Protect TTree ctor against null gDirectory
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

Successfully merging this pull request may close these issues.

2 participants