Use QProcess::nullDevice() instead of own solution
ClosedPublic

Authored by arichardson on Mar 31 2016, 5:56 PM.

Details

Summary

Correctly determine language for more -std= flags

In particular we now support e.g. gnu++1z for C++ as well as gnu99 or
iso9899:1999 and similar for C. This means that the -imacros file will
no longer define __cplusplus when parsing C code.

We also print a warning message now we couldn't determine the language
from the -std= flag

Test Plan

My files that are compiled with -std==iso9899:1999 show errors about
nullptr not being known. This happened before because __cplusplus was
defined and NULL was therefore defined to nullptr

Diff Detail

Repository
R32 KDevelop
Branch
5.0
Lint
No Linters Available
Unit
No Unit Test Coverage
arichardson updated this revision to Diff 3048.Mar 31 2016, 5:56 PM
arichardson retitled this revision from to Use QProcess::nullDevice() instead of own solution.
arichardson updated this object.
arichardson edited the test plan for this revision. (Show Details)
arichardson added reviewers: kfunk, mwolff.
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptMar 31 2016, 5:56 PM
kfunk accepted this revision.Mar 31 2016, 6:04 PM
kfunk edited edge metadata.

Thanks

This revision is now accepted and ready to land.Mar 31 2016, 6:04 PM
This revision was automatically updated to reflect the committed changes.