Don't generate trailing whitespace for IPDL CppDirective's with no |rest|

RESOLVED FIXED in mozilla34

Status

()

Core
IPC
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: mccr8, Assigned: mccr8)

Tracking

Trunk
mozilla34
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

4 years ago
Created attachment 8466545 [details] [diff] [review]
Add comment for ANDROID endif in IPDL codegen.
Attachment #8466545 - Flags: review?(bent.mozilla)
(Assignee)

Comment 2

4 years ago
Comment on attachment 8466545 [details] [diff] [review]
Add comment for ANDROID endif in IPDL codegen.

I guess I might as well fix all of the endifs at once.
Attachment #8466545 - Flags: review?(bent.mozilla)
(Assignee)

Updated

4 years ago
Summary: Add comment to ANDROID endif to avoid trailing whitespace in IPDL codegen → Don't generate trailing whitespace for IPDL CppDirective's with no |rest|
(Assignee)

Comment 3

4 years ago
Created attachment 8468610 [details] [diff] [review]
Don't generate trailing whitespace for IPDL CppDirective's with no |rest|.

try run: https://tbpl.mozilla.org/?tree=Try&rev=c2d7bf52d31d
Attachment #8468610 - Flags: review?(bent.mozilla)
Attachment #8468610 - Flags: review?(bent.mozilla) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
(Assignee)

Updated

4 years ago
Attachment #8466545 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/ead80b74189c
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.