Differential D14776

Fix all MSBuild Warnings - Category C4138: '*/' found outside of comment
ClosedPublic
Actions

Authored by volden on Aug 12 2018, 6:47 PM.

Details

Reviewers
kossebau
Group Reviewers
KDevelop
Commits
R32:e632d27c8eef: Fix all MSBuild Warnings - Category C4138: '*/' found outside of comment
Summary

This patch fixes all warnings in the windows CI build related to Category C4138: ('*/' found outside of comment)

Test Plan

Tests pass/fail as before

Diff Detail

Repository
R32 KDevelop
Branch
fixMSBUILDwarnings (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1770
Build 1788: arc lint + arc unit
volden created this revision.Aug 12 2018, 6:47 PM
Restricted Application added a project: KDevelop. · View Herald TranscriptAug 12 2018, 6:47 PM
volden requested review of this revision.Aug 12 2018, 6:47 PM
kossebau accepted this revision.Aug 13 2018, 10:00 AM
kossebau added a subscriber: kossebau.

While touching those lines, remove the space between the type and the * as well (foo */*bar*/ -> foo* /*bar*/), that follows kdevelop's general code style more closely.

This revision is now accepted and ready to land.Aug 13 2018, 10:00 AM
volden updated this revision to Diff 39590.Aug 13 2018, 12:35 PM

Updating D14776: Fix all MSBuild Warnings - Category C4138: '*/' found outside of comment

Closed by commit R32:e632d27c8eef: Fix all MSBuild Warnings - Category C4138: '*/' found outside of comment (authored by volden). · Explain WhyAug 13 2018, 12:36 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 39590

View Options

kdevplatform/shell/tests/test_projectcontroller.cpp

Loading...
View Options

plugins/debuggercommon/tests/debuggees/debugeethreads.cpp

Loading...
View Options

plugins/gdb/gdboutputwidget.cpp

Loading...