Re-Organized includes - Githubissues

ComputationalRadiationPhysics / haseongpu

HASEonGPU: High performance Amplified Spontaneous Emission on GPU

http://www.hzdr.de/crp

Other

7 stars 5 forks source link

Re-Organized includes #84

Closed slizzered closed 9 years ago

slizzered commented 9 years ago

Files now more often directly include what they need
some unnecesssary includes removed
removed custom thrust_nowarn headers, since warning no longer happens
some of the removed includes did conflict with windows, so this also helps a little with #81

I did use eclipse to get help on what to change. Since eclipse is not perfect in doing so, I checked everything manually. I might have missed something. However, it builds without warnings on my machine.

slizzered commented 9 years ago

Something I just thought of.... Old binary size: 4.9 MB New binary size after this PR is merged: 4.3 MB :dancers:

erikzenker commented 9 years ago

Ready to merge :sparkles: