From faa31be8d44ca9b62d939c49545f5d9cd527dd9f Mon Sep 17 00:00:00 2001 From: Even Rouault Date: Fri, 20 Sep 2024 02:37:43 +0200 Subject: [PATCH] PDF: rename ROUND_TO_INT_IF_CLOSE() function --- frmts/pdf/pdfdataset.cpp | 21 +++++++++------------ frmts/pdf/pdfobject.cpp | 6 +++--- frmts/pdf/pdfobject.h | 2 +- 3 files changed, 13 insertions(+), 16 deletions(-) diff --git a/frmts/pdf/pdfdataset.cpp b/frmts/pdf/pdfdataset.cpp index c0baf91be007..46044715bb65 100644 --- a/frmts/pdf/pdfdataset.cpp +++ b/frmts/pdf/pdfdataset.cpp @@ -3025,9 +3025,9 @@ static int GDALPDFParseStreamContent(const char *pszContent, INT_MAX && dfHeight / dfScaleY * DEFAULT_DPI < INT_MAX) { - double dfDPI_X = ROUND_TO_INT_IF_CLOSE( + double dfDPI_X = ROUND_IF_CLOSE( dfWidth / dfScaleX * DEFAULT_DPI, 1e-3); - double dfDPI_Y = ROUND_TO_INT_IF_CLOSE( + double dfDPI_Y = ROUND_IF_CLOSE( dfHeight / dfScaleY * DEFAULT_DPI, 1e-3); // CPLDebug("PDF", "Image %s, width = %.16g, @@ -3462,8 +3462,7 @@ void PDFDataset::GuessDPI(GDALPDFDictionary *poPageDict, int *pnBands) (poUserUnit->GetType() == PDFObjectType_Int || poUserUnit->GetType() == PDFObjectType_Real)) { - m_dfDPI = - ROUND_TO_INT_IF_CLOSE(Get(poUserUnit) * DEFAULT_DPI, 1e-5); + m_dfDPI = ROUND_IF_CLOSE(Get(poUserUnit) * DEFAULT_DPI, 1e-5); CPLDebug("PDF", "Found UserUnit in Page --> DPI = %.16g", m_dfDPI); SetMetadataItem("DPI", CPLSPrintf("%.16g", m_dfDPI)); } @@ -5398,13 +5397,11 @@ PDFDataset *PDFDataset::Open(GDALOpenInfo *poOpenInfo) ? 1e-5 : 1e-8; poDS->m_adfGeoTransform[0] = - ROUND_TO_INT_IF_CLOSE(poDS->m_adfGeoTransform[0], dfEps); - poDS->m_adfGeoTransform[1] = - ROUND_TO_INT_IF_CLOSE(poDS->m_adfGeoTransform[1]); + ROUND_IF_CLOSE(poDS->m_adfGeoTransform[0], dfEps); + poDS->m_adfGeoTransform[1] = ROUND_IF_CLOSE(poDS->m_adfGeoTransform[1]); poDS->m_adfGeoTransform[3] = - ROUND_TO_INT_IF_CLOSE(poDS->m_adfGeoTransform[3], dfEps); - poDS->m_adfGeoTransform[5] = - ROUND_TO_INT_IF_CLOSE(poDS->m_adfGeoTransform[5]); + ROUND_IF_CLOSE(poDS->m_adfGeoTransform[3], dfEps); + poDS->m_adfGeoTransform[5] = ROUND_IF_CLOSE(poDS->m_adfGeoTransform[5]); if (bUseLib.test(PDFLIB_PDFIUM)) { @@ -7208,8 +7205,8 @@ int PDFDataset::ParseMeasure(GDALPDFObject *poMeasure, double dfMediaBoxWidth, } } - x = ROUND_TO_INT_IF_CLOSE(x); - y = ROUND_TO_INT_IF_CLOSE(y); + x = ROUND_IF_CLOSE(x); + y = ROUND_IF_CLOSE(y); asGCPS[i].dfGCPX = x; asGCPS[i].dfGCPY = y; diff --git a/frmts/pdf/pdfobject.cpp b/frmts/pdf/pdfobject.cpp index d209e920181a..21c2659d6cfa 100644 --- a/frmts/pdf/pdfobject.cpp +++ b/frmts/pdf/pdfobject.cpp @@ -40,10 +40,10 @@ #include "pdfobject.h" /************************************************************************/ -/* ROUND_TO_INT_IF_CLOSE() */ +/* ROUND_IF_CLOSE() */ /************************************************************************/ -double ROUND_TO_INT_IF_CLOSE(double x, double eps) +double ROUND_IF_CLOSE(double x, double eps) { if (eps == 0.0) eps = fabs(x) < 1 ? 1e-10 : 1e-8; @@ -297,7 +297,7 @@ void GDALPDFObject::Serialize(CPLString &osStr, bool bEmitRef) { char szReal[512]; double dfRealNonRounded = GetReal(); - double dfReal = ROUND_TO_INT_IF_CLOSE(dfRealNonRounded); + double dfReal = ROUND_IF_CLOSE(dfRealNonRounded); if (dfReal >= static_cast(std::numeric_limits::min()) && dfReal <= diff --git a/frmts/pdf/pdfobject.h b/frmts/pdf/pdfobject.h index c6ffedd5943f..0021cd6d4e11 100644 --- a/frmts/pdf/pdfobject.h +++ b/frmts/pdf/pdfobject.h @@ -46,7 +46,7 @@ #define DEFAULT_DPI (72.0) #define USER_UNIT_IN_INCH (1.0 / DEFAULT_DPI) -double ROUND_TO_INT_IF_CLOSE(double x, double eps = 0); +double ROUND_IF_CLOSE(double x, double eps = 0); typedef enum {