Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(678)

Issue 4951048: Add newline at end of file to get rid of warnings compiling Chromium on the Mac.

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 3 months ago by dmac
Modified:
13 years, 3 months ago
Reviewers:
dmaclach, Stephen White
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Add newline at end of file to get rid of warnings compiling Chromium on the Mac. Chromium build wants newline at end of files. BUG=NONE TEST=BUILD

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M gpu/src/GrPathRendererChain.h View 2 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 5
dmac
PTAL
13 years, 3 months ago (2011-08-29 21:33:52 UTC) #1
Stephen White
On 2011/08/29 21:33:52, dmac wrote: > PTAL LGTM
13 years, 3 months ago (2011-08-29 21:36:31 UTC) #2
dmaclach_google.com
Would you be able to patch it in, or add me as a committer, and ...
13 years, 3 months ago (2011-08-29 21:38:54 UTC) #3
Stephen White
On 2011/08/29 21:38:54, dmaclach_google.com wrote: > Would you be able to patch it in, or ...
13 years, 3 months ago (2011-08-29 21:45:54 UTC) #4
dmaclach_google.com
13 years, 3 months ago (2011-08-29 21:47:49 UTC) #5
Thank you kindly sir.

On Mon, Aug 29, 2011 at 14:45,  <senorblanco@chromium.org> wrote:
> On 2011/08/29 21:38:54, dmaclach_google.com wrote:
>>
>> Would you be able to patch it in, or add me as a committer, and I can
>
> do it.
>
> Landed as r2188.  Closing.
>
>> Cheers,
>> Dave
>
>> On Mon, Aug 29, 2011 at 14:36,  <mailto:senorblanco@chromium.org>
>
> wrote:
>>
>> > On 2011/08/29 21:33:52, dmac wrote:
>> >>
>> >> PTAL
>> >
>> > LGTM
>> >
>> > http://codereview.appspot.com/4951048/
>> >
>
>
>
> http://codereview.appspot.com/4951048/
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b