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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Globally apply preprocessor directive coding style #8259

Merged
merged 1 commit into from Jul 3, 2020

Conversation

@techgeeknz
Copy link
Contributor

@techgeeknz techgeeknz commented Jul 2, 2020

I sed I would do this 馃槈

Command used:

find . -type f -regex '.*\.[ch]\(pp\)?\(\.[^/]*\)?' -print0 | xargs -0 sed -e's/^\([[:blank:]]\+\)[#]/#\1/g' -i
src/blitter/32bpp_anim_sse4.cpp Outdated Show resolved Hide resolved
Global; except for the 32-bit SSE blitter, which has some #DEFINEs
in not-very-nice places.
@techgeeknz techgeeknz force-pushed the techgeeknz:master_stdafx branch from c3550d7 to a48990f Jul 3, 2020
@LordAro
LordAro approved these changes Jul 3, 2020
Copy link
Member

@LordAro LordAro left a comment

That works too :)

@nielsmh nielsmh merged commit 716c883 into OpenTTD:master Jul 3, 2020
8 checks passed
8 checks passed
Commit checker
Details
OpenTTD CI Build #20200703.3 succeeded
Details
OpenTTD CI (Linux linux-amd64-clang-3.9) Linux linux-amd64-clang-3.9 succeeded
Details
OpenTTD CI (Linux linux-amd64-gcc-6) Linux linux-amd64-gcc-6 succeeded
Details
OpenTTD CI (Linux linux-i386-gcc-6) Linux linux-i386-gcc-6 succeeded
Details
OpenTTD CI (MacOS) MacOS succeeded
Details
OpenTTD CI (Windows Win32) Windows Win32 succeeded
Details
OpenTTD CI (Windows Win64) Windows Win64 succeeded
Details
@techgeeknz
Copy link
Contributor Author

@techgeeknz techgeeknz commented Jul 3, 2020

That works too :)

What problem? I don't see a problem 馃槒.

@techgeeknz techgeeknz deleted the techgeeknz:master_stdafx branch Jul 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can鈥檛 perform that action at this time.