diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2016-11-21 13:50:32 -0500 |
|---|---|---|
| committer | dan sinclair <dsinclair@chromium.org> | 2016-11-21 18:51:10 +0000 |
| commit | 85c8e7f788512ce5b5b43cd816e8091b57c3d5e7 (patch) | |
| tree | e727d747684b9db2632ef250b55046cfce2b55a3 /fpdfsdk/javascript/app.h | |
| parent | 7daa6fb286d1b5d65f99521f3314083e86d93e3f (diff) | |
| download | pdfium-85c8e7f788512ce5b5b43cd816e8091b57c3d5e7.tar.xz | |
Fixup lint flags.
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>
Diffstat (limited to 'fpdfsdk/javascript/app.h')
| -rw-r--r-- | fpdfsdk/javascript/app.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fpdfsdk/javascript/app.h b/fpdfsdk/javascript/app.h index 32e2aaf28c..e8c72417bb 100644 --- a/fpdfsdk/javascript/app.h +++ b/fpdfsdk/javascript/app.h @@ -18,7 +18,7 @@ class GlobalTimer; class TimerObj : public CJS_EmbedObj { public: - TimerObj(CJS_Object* pJSObject); + explicit TimerObj(CJS_Object* pJSObject); ~TimerObj() override; void SetTimer(GlobalTimer* pTimer); @@ -30,7 +30,7 @@ class TimerObj : public CJS_EmbedObj { class CJS_TimerObj : public CJS_Object { public: - CJS_TimerObj(v8::Local<v8::Object> pObject) : CJS_Object(pObject) {} + explicit CJS_TimerObj(v8::Local<v8::Object> pObject) : CJS_Object(pObject) {} ~CJS_TimerObj() override {} DECLARE_JS_CLASS(); @@ -38,7 +38,7 @@ class CJS_TimerObj : public CJS_Object { class app : public CJS_EmbedObj { public: - app(CJS_Object* pJSObject); + explicit app(CJS_Object* pJSObject); ~app() override; bool activeDocs(IJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError); |