Add NOLINTNEXTLINE missed in prior CL. Should have been part of https://pdfium-review.googlesource.com/c/pdfium/+/100680 Change-Id: I4ba03ec2e3bb9ead64facc0745fe446422b4562c Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100682 Commit-Queue: Lei Zhang <thestig@chromium.org> Auto-Submit: Tom Sepez <tsepez@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
diff --git a/fpdfsdk/fpdf_structtree.cpp b/fpdfsdk/fpdf_structtree.cpp index b0029cc..76a30be 100644 --- a/fpdfsdk/fpdf_structtree.cpp +++ b/fpdfsdk/fpdf_structtree.cpp
@@ -163,9 +163,10 @@ const CPDF_Array* array = attr_obj->AsArray(); if (index < 0 || static_cast<size_t>(index) >= array->size()) return nullptr; + // TODO(tsepez): should embedder take a reference here? - return FPDFStructElementAttrFromCPDFDictionary( - array->GetDictAt(index).Get()); + // Unretained reference in public API. NOLINTNEXTLINE + return FPDFStructElementAttrFromCPDFDictionary(array->GetDictAt(index)); } return nullptr; }