diff options
| author | Lei Zhang <thestig@chromium.org> | 2018-05-09 22:02:32 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-05-09 22:02:32 +0000 |
| commit | 7e28208d26764438bef62e051d2e1fed13e1e0ec (patch) | |
| tree | 781d3e250a4fd3af3eb16da299b56938b21a7414 /core/fpdfapi/parser/cpdf_dictionary.cpp | |
| parent | 52489ee8114cf476342231be610e75b324290920 (diff) | |
| download | pdfium-7e28208d26764438bef62e051d2e1fed13e1e0ec.tar.xz | |
Add proper const/non-const versions of CPDF_Object methods.
Instead of having const methods that return non-const pointers.
BUG=pdfium:234
Change-Id: I598e9b4f267f702e2e71001cfe3aa3c9e9d8c12f
Reviewed-on: https://pdfium-review.googlesource.com/32182
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'core/fpdfapi/parser/cpdf_dictionary.cpp')
| -rw-r--r-- | core/fpdfapi/parser/cpdf_dictionary.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/core/fpdfapi/parser/cpdf_dictionary.cpp b/core/fpdfapi/parser/cpdf_dictionary.cpp index b3195b58af..29571d6c51 100644 --- a/core/fpdfapi/parser/cpdf_dictionary.cpp +++ b/core/fpdfapi/parser/cpdf_dictionary.cpp @@ -42,10 +42,12 @@ CPDF_Object::Type CPDF_Dictionary::GetType() const { return DICTIONARY; } -CPDF_Dictionary* CPDF_Dictionary::GetDict() const { - // The method should be made non-const if we want to not be const. - // See bug #234. - return const_cast<CPDF_Dictionary*>(this); +CPDF_Dictionary* CPDF_Dictionary::GetDict() { + return this; +} + +const CPDF_Dictionary* CPDF_Dictionary::GetDict() const { + return this; } bool CPDF_Dictionary::IsDictionary() const { |