Issue 1026843002: Fix missing include for std::min - Code Review

Chromium Code Reviews

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

(418)

My Issues | Starred Open | Closed | All

Issue 1026843002: Fix missing include for std::min (Closed)

Created:
5 years, 9 months ago by Tom Sepez

Modified:
5 years, 9 months ago

Reviewers:
Lei Zhang

CC:
pdfium-reviews_googlegroups.com

Base URL:
https://pdfium.googlesource.com/pdfium.git@master

Target Ref:
refs/heads/master

Visibility:
Public.

More Reviews

Description

Fix missing include for std::min Build broken on trybot at http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel/builds/66012/steps/compile%20%28with%20patch%29/logs/stdio R=thestig@chromium.org TBR=thestig@chromium.org Committed: https://pdfium.googlesource.com/pdfium/+/ae702b094ba8fda303b4c7e6ddafb3ebae6db7b6

Patch Set 1 #

Created: 5 years, 9 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+1 line, -0 lines)			Patch
	M	core/src/fxcodec/codec/fx_codec_jpx_opj.cpp	View		1 chunk	+1 line, -0 lines	0 comments	Download

Messages

Total messages: 5 (0 generated)

Expand Messages | Collapse Messages

Tom Sepez

Lei, trying to fix the damage I introduced the other day. Thanks.

5 years, 9 months ago (2015-03-20 16:57:31 UTC) #2

Tom Sepez

Committed patchset #1 (id:1) manually as ae702b094ba8fda303b4c7e6ddafb3ebae6db7b6 (presubmit successful).

5 years, 9 months ago (2015-03-20 17:53:07 UTC) #4

Message was sent while issue was closed.

lgtm

Expand Messages | Collapse Messages

Powered by Google App Engine

This is Rietveld 408576698