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

Remove tab characters from C source files. #9397

Closed
wants to merge 2 commits into from

Conversation

paulidale
Copy link
Contributor

Some have been creeping into the source code.
Also remove end of line spaces.

  • tests are added or updated

Copy link
Member

@InfoHunter InfoHunter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am surprised there are so many tabs...

@paulidale
Copy link
Contributor Author

There are more tabs in the assembly, I didn't touch them.

@paulidale paulidale added the branch: master Merge to master branch label Jul 17, 2019
@paulidale
Copy link
Contributor Author

Merged to master, thanks.

@paulidale paulidale closed this Jul 17, 2019
@paulidale paulidale deleted the tab branch July 17, 2019 02:58
levitte pushed a commit that referenced this pull request Jul 17, 2019
Some have been creeping into the source code.

Reviewed-by: Paul Yang <yang.yang@baishancloud.com>
(Merged from #9397)
levitte pushed a commit that referenced this pull request Jul 17, 2019
Reviewed-by: Paul Yang <yang.yang@baishancloud.com>
(Merged from #9397)
@InfoHunter
Copy link
Member

Errr, I just found my email address in the commit stays incorrect. Should be something like:

Reviewed-by: Paul Yang <kaishen.yy@alipay.com>

instead of:

Reviewed-by: Paul Yang <yang.yang@baishancloud.com>

I dropped an email to @levitte yesterday about the information update due to a job change of mine. It seems it's better if I send the information to the @openssl/omc as well...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch: master Merge to master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants