From 5f244a97085d0f95d65e2cc213efaf0e71ae691d Mon Sep 17 00:00:00 2001 From: Miklos Vajna Date: Thu, 31 Aug 2017 09:41:33 +0200 Subject: Convert 0 to nullptr where it's used as a pointer value To make the code easier to read, because this means the remaining zeros all mean numbers. Change-Id: Iaacefbd691007ec6ffe360b810df5997952cc87e --- src/conv/raw/fh2raw.cpp | 2 +- src/conv/svg/fh2svg.cpp | 2 +- src/conv/text/fh2text.cpp | 2 +- src/lib/FHCollector.cpp | 156 +++++++++++++++++++++---------------------- src/lib/FHInternalStream.cpp | 4 +- src/lib/FHInternalStream.h | 6 +- src/lib/FHParser.cpp | 8 +-- 7 files changed, 90 insertions(+), 90 deletions(-) diff --git a/src/conv/raw/fh2raw.cpp b/src/conv/raw/fh2raw.cpp index 86f7c37..24ab7d7 100644 --- a/src/conv/raw/fh2raw.cpp +++ b/src/conv/raw/fh2raw.cpp @@ -55,7 +55,7 @@ int printVersion() int main(int argc, char *argv[]) { bool printIndentLevel = false; - char *file = 0; + char *file = nullptr; if (argc < 2) return printUsage(); diff --git a/src/conv/svg/fh2svg.cpp b/src/conv/svg/fh2svg.cpp index 64a4286..05531e0 100644 --- a/src/conv/svg/fh2svg.cpp +++ b/src/conv/svg/fh2svg.cpp @@ -53,7 +53,7 @@ int main(int argc, char *argv[]) if (argc < 2) return printUsage(); - char *file = 0; + char *file = nullptr; for (int i = 1; i < argc; i++) { diff --git a/src/conv/text/fh2text.cpp b/src/conv/text/fh2text.cpp index 2cebf10..6fe0f9d 100644 --- a/src/conv/text/fh2text.cpp +++ b/src/conv/text/fh2text.cpp @@ -53,7 +53,7 @@ int main(int argc, char *argv[]) if (argc < 2) return printUsage(); - char *file = 0; + char *file = nullptr; for (int i = 1; i < argc; i++) { diff --git a/src/lib/FHCollector.cpp b/src/lib/FHCollector.cpp index 0f0cf36..60d8b0a 100644 --- a/src/lib/FHCollector.cpp +++ b/src/lib/FHCollector.cpp @@ -2179,7 +2179,7 @@ const std::vector *libfreehand::FHCollector::_findListElements(unsigne std::map::const_iterator iter = m_lists.find(id); if (iter != m_lists.end()) return &(iter->second.m_elements); - return 0; + return nullptr; } @@ -2695,371 +2695,371 @@ void libfreehand::FHCollector::_appendPatternLine(librevenge::RVNGPropertyList & const libfreehand::FHPath *libfreehand::FHCollector::_findPath(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_paths.find(id); if (iter != m_paths.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHNewBlend *libfreehand::FHCollector::_findNewBlend(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_newBlends.find(id); if (iter != m_newBlends.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHGroup *libfreehand::FHCollector::_findGroup(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_groups.find(id); if (iter != m_groups.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHGroup *libfreehand::FHCollector::_findClipGroup(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_clipGroups.find(id); if (iter != m_clipGroups.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHCompositePath *libfreehand::FHCollector::_findCompositePath(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_compositePaths.find(id); if (iter != m_compositePaths.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHPathText *libfreehand::FHCollector::_findPathText(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_pathTexts.find(id); if (iter != m_pathTexts.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHTextObject *libfreehand::FHCollector::_findTextObject(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_textObjects.find(id); if (iter != m_textObjects.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHTransform *libfreehand::FHCollector::_findTransform(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_transforms.find(id); if (iter != m_transforms.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHTEffect *libfreehand::FHCollector::_findTEffect(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_tEffects.find(id); if (iter != m_tEffects.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHParagraph *libfreehand::FHCollector::_findParagraph(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_paragraphs.find(id); if (iter != m_paragraphs.end()) return &(iter->second); - return 0; + return nullptr; } const std::vector *libfreehand::FHCollector::_findTabTable(unsigned id) { if (!id) - return 0; + return nullptr; std::map >::const_iterator iter = m_tabs.find(id); if (iter != m_tabs.end()) return &(iter->second); - return 0; + return nullptr; } const std::vector *libfreehand::FHCollector::_findTStringElements(unsigned id) { if (!id) - return 0; + return nullptr; std::map >::const_iterator iter = m_tStrings.find(id); if (iter != m_tStrings.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHPropList *libfreehand::FHCollector::_findPropList(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_propertyLists.find(id); if (iter != m_propertyLists.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHGraphicStyle *libfreehand::FHCollector::_findGraphicStyle(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_graphicStyles.find(id); if (iter != m_graphicStyles.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHBasicFill *libfreehand::FHCollector::_findBasicFill(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_basicFills.find(id); if (iter != m_basicFills.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHLinearFill *libfreehand::FHCollector::_findLinearFill(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_linearFills.find(id); if (iter != m_linearFills.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHLensFill *libfreehand::FHCollector::_findLensFill(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_lensFills.find(id); if (iter != m_lensFills.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHRadialFill *libfreehand::FHCollector::_findRadialFill(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_radialFills.find(id); if (iter != m_radialFills.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHTileFill *libfreehand::FHCollector::_findTileFill(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_tileFills.find(id); if (iter != m_tileFills.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHPatternFill *libfreehand::FHCollector::_findPatternFill(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_patternFills.find(id); if (iter != m_patternFills.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHLinePattern *libfreehand::FHCollector::_findLinePattern(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_linePatterns.find(id); if (iter != m_linePatterns.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHPath *libfreehand::FHCollector::_findArrowPath(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_arrowPaths.find(id); if (iter != m_arrowPaths.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHBasicLine *libfreehand::FHCollector::_findBasicLine(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_basicLines.find(id); if (iter != m_basicLines.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHCustomProc *libfreehand::FHCollector::_findCustomProc(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_customProcs.find(id); if (iter != m_customProcs.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHPatternLine *libfreehand::FHCollector::_findPatternLine(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_patternLines.find(id); if (iter != m_patternLines.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHRGBColor *libfreehand::FHCollector::_findRGBColor(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_rgbColors.find(id); if (iter != m_rgbColors.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHTintColor *libfreehand::FHCollector::_findTintColor(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_tints.find(id); if (iter != m_tints.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHDisplayText *libfreehand::FHCollector::_findDisplayText(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_displayTexts.find(id); if (iter != m_displayTexts.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHImageImport *libfreehand::FHCollector::_findImageImport(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_images.find(id); if (iter != m_images.end()) return &(iter->second); - return 0; + return nullptr; } const librevenge::RVNGBinaryData *libfreehand::FHCollector::_findData(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_data.find(id); if (iter != m_data.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHSymbolClass *libfreehand::FHCollector::_findSymbolClass(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_symbolClasses.find(id); if (iter != m_symbolClasses.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHSymbolInstance *libfreehand::FHCollector::_findSymbolInstance(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_symbolInstances.find(id); if (iter != m_symbolInstances.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FHFilterAttributeHolder *libfreehand::FHCollector::_findFilterAttributeHolder(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_filterAttributeHolders.find(id); if (iter != m_filterAttributeHolders.end()) return &(iter->second); - return 0; + return nullptr; } const std::vector *libfreehand::FHCollector::_findMultiColorList(unsigned id) { if (!id) - return 0; + return nullptr; std::map >::const_iterator iter = m_multiColorLists.find(id); if (iter != m_multiColorLists.end()) return &(iter->second); - return 0; + return nullptr; } const double *libfreehand::FHCollector::_findOpacityFilter(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_opacityFilters.find(id); if (iter != m_opacityFilters.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FWShadowFilter *libfreehand::FHCollector::_findFWShadowFilter(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_shadowFilters.find(id); if (iter != m_shadowFilters.end()) return &(iter->second); - return 0; + return nullptr; } const libfreehand::FWGlowFilter *libfreehand::FHCollector::_findFWGlowFilter(unsigned id) { if (!id) - return 0; + return nullptr; std::map::const_iterator iter = m_glowFilters.find(id); if (iter != m_glowFilters.end()) return &(iter->second); - return 0; + return nullptr; } unsigned libfreehand::FHCollector::_findStrokeId(const libfreehand::FHGraphicStyle &graphicStyle) @@ -3105,10 +3105,10 @@ const libfreehand::FHFilterAttributeHolder *libfreehand::FHCollector::_findFilte { unsigned listId = graphicStyle.m_attrId; if (!listId) - return 0; + return nullptr; std::map::const_iterator iter = m_lists.find(listId); if (iter == m_lists.end()) - return 0; + return nullptr; for (unsigned i = 0; i < iter->second.m_elements.size(); ++i) { const FHFilterAttributeHolder *attributeHolder = _findFilterAttributeHolder(iter->second.m_elements[i]); @@ -3116,7 +3116,7 @@ const libfreehand::FHFilterAttributeHolder *libfreehand::FHCollector::_findFilte if (attributeHolder) return attributeHolder; } - return 0; + return nullptr; } diff --git a/src/lib/FHInternalStream.cpp b/src/lib/FHInternalStream.cpp index e66db1b..4c30fa9 100644 --- a/src/lib/FHInternalStream.cpp +++ b/src/lib/FHInternalStream.cpp @@ -95,7 +95,7 @@ const unsigned char *libfreehand::FHInternalStream::read(unsigned long numBytes, numBytesRead = 0; if (numBytes == 0) - return 0; + return nullptr; unsigned numBytesToRead; @@ -107,7 +107,7 @@ const unsigned char *libfreehand::FHInternalStream::read(unsigned long numBytes, numBytesRead = numBytesToRead; // about as paranoid as we can be.. if (numBytesToRead == 0) - return 0; + return nullptr; long oldOffset = m_offset; m_offset += numBytesToRead; diff --git a/src/lib/FHInternalStream.h b/src/lib/FHInternalStream.h index 5f3bad5..581295e 100644 --- a/src/lib/FHInternalStream.h +++ b/src/lib/FHInternalStream.h @@ -33,7 +33,7 @@ public: } const char *subStreamName(unsigned) override { - return 0; + return nullptr; } bool existsSubStream(const char *) override { @@ -41,11 +41,11 @@ public: } librevenge::RVNGInputStream *getSubStreamByName(const char *) override { - return 0; + return nullptr; } librevenge::RVNGInputStream *getSubStreamById(unsigned) override { - return 0; + return nullptr; } const unsigned char *read(unsigned long numBytes, unsigned long &numBytesRead) override; int seek(long offset, librevenge::RVNG_SEEK_TYPE seekType) override; diff --git a/src/lib/FHParser.cpp b/src/lib/FHParser.cpp index 982118e..93488c7 100644 --- a/src/lib/FHParser.cpp +++ b/src/lib/FHParser.cpp @@ -43,7 +43,7 @@ static int getTokenId(const char *name) const char *getTokenName(int tokenId) { if (tokenId >= FH_TOKEN_COUNT) - return 0; + return nullptr; const fhtoken *currentToken = wordlist; while (currentToken != wordlist+sizeof(wordlist)/sizeof(*wordlist)) { @@ -51,7 +51,7 @@ const char *getTokenName(int tokenId) return currentToken->name; ++currentToken; } - return 0; + return nullptr; } #endif @@ -59,8 +59,8 @@ const char *getTokenName(int tokenId) } // anonymous namespace libfreehand::FHParser::FHParser() - : m_input(0), m_collector(0), m_version(-1), m_dictionary(), - m_records(), m_currentRecord(0), m_pageInfo(), m_colorTransform(0) + : m_input(nullptr), m_collector(nullptr), m_version(-1), m_dictionary(), + m_records(), m_currentRecord(0), m_pageInfo(), m_colorTransform(nullptr) { cmsHPROFILE inProfile = cmsOpenProfileFromMem(CMYK_icc, sizeof(CMYK_icc)/sizeof(CMYK_icc[0])); cmsHPROFILE outProfile = cmsCreate_sRGBProfile(); -- cgit v1.2.3