diff options
author | Tom Sepez <tsepez@chromium.org> | 2018-08-17 19:28:52 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-08-17 19:28:52 +0000 |
commit | 3a6d058740a9908a06efb9e9362df01acdee6ff8 (patch) | |
tree | 9bc645b0cb3442749458fc63e2feb04c0cb813ef /fxjs/cjs_return.h | |
parent | 0a2328e0eff7a11aa49b7da8f013e658153a7b1a (diff) | |
download | pdfium-3a6d058740a9908a06efb9e9362df01acdee6ff8.tar.xz |
Rename CJS_Return to CJS_Result.
"Return" is a verb, and "return" is a reserved-word at that,
so avoid using it as part of a class name.
Fully mechanical change apart from rename.
Change-Id: I120e453e8ba001c4ab74a39e2da6aa6eb590835f
Reviewed-on: https://pdfium-review.googlesource.com/40532
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fxjs/cjs_return.h')
-rw-r--r-- | fxjs/cjs_return.h | 57 |
1 files changed, 0 insertions, 57 deletions
diff --git a/fxjs/cjs_return.h b/fxjs/cjs_return.h deleted file mode 100644 index 99a2af60a6..0000000000 --- a/fxjs/cjs_return.h +++ /dev/null @@ -1,57 +0,0 @@ -// Copyright 2017 PDFium Authors. All rights reserved. -// Use of this source code is governed by a BSD-style license that can be -// found in the LICENSE file. - -// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com - -#ifndef FXJS_CJS_RETURN_H_ -#define FXJS_CJS_RETURN_H_ - -#include "fxjs/cfxjs_engine.h" -#include "fxjs/js_resources.h" -#include "third_party/base/optional.h" - -class CJS_Return { - public: - // Wrap constructors with static methods so we can apply WARN_UNUSED_RESULT, - // otherwise we can't catch places where someone mistakenly writes: - // - // if (error) - // CJS_Return(JS_ERROR_CODE); - // - // instead of - // - // if (error) - // return CJS_Return(JS_ERROR_CODE); - // - static CJS_Return Success() WARN_UNUSED_RESULT { return CJS_Return(); } - static CJS_Return Success(v8::Local<v8::Value> value) WARN_UNUSED_RESULT { - return CJS_Return(value); - } - static CJS_Return Failure(const WideString& str) WARN_UNUSED_RESULT { - return CJS_Return(str); - } - static CJS_Return Failure(JSMessage id) WARN_UNUSED_RESULT { - return CJS_Return(id); - } - - CJS_Return(const CJS_Return&); - ~CJS_Return(); - - bool HasError() const { return error_.has_value(); } - WideString Error() const { return error_.value(); } - - bool HasReturn() const { return !return_.IsEmpty(); } - v8::Local<v8::Value> Return() const { return return_; } - - private: - CJS_Return(); // Successful but empty return. - explicit CJS_Return(v8::Local<v8::Value>); // Successful return with value. - explicit CJS_Return(const WideString&); // Error with custom message. - explicit CJS_Return(JSMessage id); // Error with stock message. - - Optional<WideString> error_; - v8::Local<v8::Value> return_; -}; - -#endif // FXJS_CJS_RETURN_H_ |