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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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.