Fix pragma once problem with process class
ClosedPublic

Authored by patrickelectric on Feb 6 2019, 11:39 PM.

Details

Summary

Signed-off-by: Patrick José Pereira <patrickelectric@gmail.com>

Diff Detail

Repository
R871 DrKonqi
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 6 2019, 11:39 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
patrickelectric requested review of this revision.Feb 6 2019, 11:39 PM
mart added a subscriber: mart.Feb 9 2019, 9:38 AM

needs better commit message

Is it because process.h exists somewhere else?

Update commit message

@mart commit message updated, @alexeymin I believe that you'll find the answer for this question in my new commit message :)

Also, this PR is really important to help Craft since the patch command used to apply .diffs/patchs can't move/rename files.
This patch needs to be added manually to do the build.

@mart commit message updated, @alexeymin I believe that you'll find the answer for this question in my new commit message :)

It hasn't updated what phab shows.

@mart commit message updated, @alexeymin I believe that you'll find the answer for this question in my new commit message :)

It hasn't updated what phab shows.

Click in commit and after that in show more.

Your comment doesn't match what the internet says about "#pragma once" (though it is a non-standard and some compilers are weird)
Two files with the same filename should be included twice.

But if this works...meh, go for it.

tcanabrava accepted this revision.Feb 10 2019, 9:08 PM
This revision is now accepted and ready to land.Feb 10 2019, 9:08 PM
This revision was automatically updated to reflect the committed changes.