pdfium.git - Unnamed repository; edit this file 'description' to name the repository.

Age	Commit message (Collapse)	Author
2017-05-19	Add test duplicate check in presubmit	Nicolas Pena
	This CL adds a presubmit check to avoid adding both .in and .pdf file to javascript and pixel tests. Change-Id: If2f252d20c3bfd3f9cd5963bb3428b57f6bee1b5 Reviewed-on: https://pdfium-review.googlesource.com/5710 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Nicolás Peña <npm@chromium.org>
2016-11-21	Check include order on cpp files as well	dsinclair
	The check for include sorting was missing the cpp extension so was not running against any cpp files on upload. Review-Url: https://codereview.chromium.org/2522673002
2016-11-21	Fixup lint flags.	Dan Sinclair
	The -build/include setting was masking out build/include_what_you_use. This CL restores them, fixes any build errors, and adds NOLINT as needed. As well, the runtime/explicit and runtime/printf flags are aslo enabled and NOLINT'd. lint cleanups Change-Id: Ib013b3eb29c8d0e48cad74c5df9028684130719f Reviewed-on: https://pdfium-review.googlesource.com/2030 Reviewed-by: Tom Sepez <tsepez@chromium.org>
2016-09-13	Sort include entries.	dsinclair
	This CL updates all of the includes to be correctly sorted. A PRESUBMIT warning is added (from chromium) that will warn if the includes are in the wrong order on upload. Review-Url: https://codereview.chromium.org/2337293002
2016-04-05	Better error for checkdeps.	dsinclair
	If the checkdeps script can not be found the following error will be issued instead of a stacktrace: Presubmit ERRORS Unable to run checkdeps, does pdfium/buildtools/checkdeps exist? BUG=chromium:600043 Review URL: https://codereview.chromium.org/1860803003
2016-03-14	Add missing DEPS files; enable checkdeps.	Dan Sinclair
	This CL adds the needed DEPS files to make checkdeps pass correctly. The checkdeps PRESUBMIT method has been copied from Chromium and is enabled on CL upload. R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1801923002 .
2016-03-09	Review and cleanup lint warnings.	Dan Sinclair
	This CL goes through the remaining list of list warnings and records why they are currently blacklisted, or fixes and enables them. R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1773733002 .
2016-03-08	Update cast to fix lint warning.	Dan Sinclair
	The current cast confused runtime/casting. Update to use static_cast and enable the lint warning. BUG=pdfium:426 R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1770253003 .
2016-03-01	Fix and enable lint checks.	Dan Sinclair
	This CL fixes and enables: * readability/namespace * readability/multiline_string * readability/multiline_comment * readability/inheritance * readability/function * readability/braces R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1747123002 .
2016-02-24	Fixing whitespace lint errors.	Dan Sinclair
	This CL enables several of the diabled whitelist/* lint checks. R=thestig@chromium.org, tsepez@chromium.org Review URL: https://codereview.chromium.org/1730553002 .
2016-02-22	Change presubmit lint check to a blacklist.	dan sinclair
	This CL changes the possible lint flags to a blacklist which can be burned down. R=thestig@chromium.org Review URL: https://codereview.chromium.org/1725573003 .
2016-02-22	Fixing include guards, and presubmit.	Dan Sinclair
	This update fixes all include guards so cpplint build/header_guard completes without error. A presubmit check is added to make sure the guards stay clean. BUG=pdfium:65 R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1719083002 .
2015-08-06	XFA: clang-format all pdfium code, again.	Nico Weber
	Also add a presubmit that checks for this so I don't have to keep doing it. No behavior change. Generated by: find . -name '.cpp' -o -name '.h' \| \ grep -E -v 'third_party\|thirdparties\|lpng_v163\|tiff_v403' \| \ xargs ../../buildtools/mac/clang-format -i Then manually merged https://codereview.chromium.org/1269223002/ See thread "tabs vs spaces" on pdfium@googlegroups.com for discussion. BUG=none R=thestig@chromium.org Review URL: https://codereview.chromium.org/1277043002 .