Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(185)

Issue 426763004: Add missing braces for aggregate initializers. (Closed)

Created:
6 years, 4 months ago by Nico
Modified:
6 years, 4 months ago
Reviewers:
Bo Xu
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Visibility:
Public.

Description

Add missing braces for aggregate initializers. No intended behavior change. BUG=pdfium:29 R=bo_xu@foxitsoftware.com Committed: https://pdfium.googlesource.com/pdfium/+/96608b3

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -12 lines) Patch
M fpdfsdk/src/javascript/util.cpp View 2 chunks +12 lines, -12 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Nico
6 years, 4 months ago (2014-07-30 21:59:03 UTC) #1
Nico
6 years, 4 months ago (2014-07-30 22:00:03 UTC) #2
Bo Xu
On 2014/07/30 22:00:03, Nico (away) wrote: lgtm
6 years, 4 months ago (2014-07-30 23:45:29 UTC) #3
Nico
6 years, 4 months ago (2014-07-31 01:59:41 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r96608b3 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698