Page Menu
Home
Phabricator
This is an archive of the discontinued LLVM Phabricator instance.
Paths
Table of Contents
t
-
lib/Format/
-
Format/
-
SortJavaScriptImports.cpp
Hide Panel
f
Keyboard Reference
?
Differential
D25599
Removed duplicate header include
Closed
Public
Authored by
krasimir
on Oct 14 2016, 2:34 AM.
Download Raw Diff
Details
Reviewers
ioeric
Commits
rG735f3a26be47: Removed duplicate header include
rC284228: Removed duplicate header include
rL284228: Removed duplicate header include
Diff Detail
Build Status
Buildable 463
Build 463: arc lint + arc unit
Event Timeline
krasimir
updated this revision to
Diff 74635
.
Oct 14 2016, 2:34 AM
krasimir
retitled this revision from
to
Removed duplicate header include
.
krasimir
updated this object.
Herald
added a subscriber:
klimek
.
·
View Herald Transcript
Oct 14 2016, 2:34 AM
krasimir
abandoned this revision.
Oct 14 2016, 2:36 AM
krasimir
updated this revision to
Diff 74637
.
Oct 14 2016, 2:41 AM
Comment Actions
Updated nothing; my first revision!
krasimir
added a reviewer:
ioeric
.
Oct 14 2016, 2:43 AM
ioeric
accepted this revision.
Oct 14 2016, 4:33 AM
ioeric
edited edge metadata.
Comment Actions
LGTM
This revision is now accepted and ready to land.
Oct 14 2016, 4:33 AM
ioeric
added a comment.
Oct 14 2016, 4:34 AM
Comment Actions
Would you like this to be submitted?
krasimir
added a comment.
Oct 14 2016, 4:50 AM
Comment Actions
Yes please!
Closed by commit
rL284228: Removed duplicate header include
(authored by
ioeric
).
·
Explain Why
Oct 14 2016, 4:57 AM
This revision was automatically updated to reflect the committed changes.