From aca3efc9c0d8750b7f642a23eea1a665d7a1af5c Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Fri, 16 Mar 2018 20:27:04 +0000 Subject: Fix methods that have definition with different parameter names. Found by ClangTidy. BUG=pdfium:1039 Change-Id: I2bb9256a63d5459e959c1a3228418073245d7a86 Reviewed-on: https://pdfium-review.googlesource.com/28731 Reviewed-by: Henrique Nakashima Commit-Queue: Lei Zhang --- fpdfsdk/fpdf_ppo.cpp | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'fpdfsdk/fpdf_ppo.cpp') diff --git a/fpdfsdk/fpdf_ppo.cpp b/fpdfsdk/fpdf_ppo.cpp index 1c9550a54a..ceceacf01d 100644 --- a/fpdfsdk/fpdf_ppo.cpp +++ b/fpdfsdk/fpdf_ppo.cpp @@ -60,11 +60,11 @@ class NupState { // use. std::pair ConvertPageOrder() const; - // Given the |subx| and |suby| subpage position within a page, and a source + // Given the |subX| and |subY| subpage position within a page, and a source // page with dimensions of |pagesize|, calculate the sub page's origin and // scale. - NupPageSettings CalculatePageEdit(size_t subx, - size_t suby, + NupPageSettings CalculatePageEdit(size_t subX, + size_t subY, const CFX_SizeF& pagesize) const; const CFX_SizeF m_destPageSize; @@ -103,12 +103,12 @@ std::pair NupState::ConvertPageOrder() const { return {subX, subY}; } -NupPageSettings NupState::CalculatePageEdit(size_t subXPos, - size_t subYPos, +NupPageSettings NupState::CalculatePageEdit(size_t subX, + size_t subY, const CFX_SizeF& pagesize) const { NupPageSettings settings; - settings.subPageStartPoint.x = subXPos * m_subPageSize.width; - settings.subPageStartPoint.y = subYPos * m_subPageSize.height; + settings.subPageStartPoint.x = subX * m_subPageSize.width; + settings.subPageStartPoint.y = subY * m_subPageSize.height; const float xScale = m_subPageSize.width / pagesize.width; const float yScale = m_subPageSize.height / pagesize.height; -- cgit v1.2.3