Commit 64a9c5c7 authored by Claudio Valerio's avatar Claudio Valerio

removing warning act 4 not very proud of changes

parent 0e3b8447
...@@ -57,7 +57,7 @@ bool ASCIIHexDecode::decode(std::string & encoded) ...@@ -57,7 +57,7 @@ bool ASCIIHexDecode::decode(std::string & encoded)
for(int i = 0;i<len;i++) for(int i = 0;i<len;i++)
{ {
unsigned char ch = encoded[i]; unsigned char ch = encoded[i];
if( WHITESPACES.find(ch) != -1 ) if((int) WHITESPACES.find(ch) != -1 )
{ {
continue; continue;
} }
......
...@@ -42,7 +42,7 @@ namespace merge_lib ...@@ -42,7 +42,7 @@ namespace merge_lib
{ {
unsigned int startOfParent = content.find("/Parent"); unsigned int startOfParent = content.find("/Parent");
unsigned int endOfParent = content.find(" R", startOfParent); unsigned int endOfParent = content.find(" R", startOfParent);
if(startOfParent == -1) if((int)startOfParent == -1)
break; break;
std::vector <Object *> parents = parent->getChildrenByBounds(startOfParent, endOfParent); std::vector <Object *> parents = parent->getChildrenByBounds(startOfParent, endOfParent);
if(parents.size() != 1) if(parents.size() != 1)
...@@ -50,7 +50,7 @@ namespace merge_lib ...@@ -50,7 +50,7 @@ namespace merge_lib
parent = parents[0]; parent = parents[0];
std::string parentContent = parent->getObjectContent(); std::string parentContent = parent->getObjectContent();
unsigned int startOfMediaBox = parentContent.find(_handlerName); unsigned int startOfMediaBox = parentContent.find(_handlerName);
if(startOfMediaBox == -1) if((int)startOfMediaBox == -1)
{ {
content = parentContent; content = parentContent;
continue; continue;
......
...@@ -36,8 +36,8 @@ void AnnotsHandler::_processObjectContent(unsigned int startOfPageElement) ...@@ -36,8 +36,8 @@ void AnnotsHandler::_processObjectContent(unsigned int startOfPageElement)
{ {
Object * child = _annotations[0]; Object * child = _annotations[0];
std::string childContent = child->getObjectContent(); std::string childContent = child->getObjectContent();
if( Parser::findToken(childContent,"/Rect") == -1 && if((int) Parser::findToken(childContent,"/Rect") == -1 &&
Parser::findToken(childContent,"/Subtype") == -1 ) (int)Parser::findToken(childContent,"/Subtype") == -1 )
{ {
// this was not Annotation but reference to array // this was not Annotation but reference to array
// of annotations // of annotations
......
...@@ -63,12 +63,12 @@ std::vector <Decoder * > Filter::_getDecoders() ...@@ -63,12 +63,12 @@ std::vector <Decoder * > Filter::_getDecoders()
while(1) while(1)
{ {
startOfDecoder = streamHeader.find("/", startOfDecoder); startOfDecoder = streamHeader.find("/", startOfDecoder);
if(startOfDecoder == -1) if((int)startOfDecoder == -1)
break; break;
else else
++startOfDecoder; ++startOfDecoder;
unsigned int endOfDecoder = streamHeader.find_first_of(whitespacesAndDelimeters, startOfDecoder); unsigned int endOfDecoder = streamHeader.find_first_of(whitespacesAndDelimeters, startOfDecoder);
if(endOfDecoder == -1) if((int)endOfDecoder == -1)
break; break;
std::map<std::string, Decoder *>::iterator foundDecoder = std::map<std::string, Decoder *>::iterator foundDecoder =
_allDecoders.find(streamHeader.substr(startOfDecoder, endOfDecoder - startOfDecoder)); _allDecoders.find(streamHeader.substr(startOfDecoder, endOfDecoder - startOfDecoder));
......
...@@ -59,14 +59,14 @@ FilterPredictor::~FilterPredictor() ...@@ -59,14 +59,14 @@ FilterPredictor::~FilterPredictor()
std::string FilterPredictor::getDictionaryContentStr(std::string & in, size_t &pos ) std::string FilterPredictor::getDictionaryContentStr(std::string & in, size_t &pos )
{ {
size_t beg = in.find(DICT_START_TOKEN,pos); size_t beg = in.find(DICT_START_TOKEN,pos);
if( beg == -1 ) if((int) beg == -1 )
{ {
return ""; return "";
} }
beg += DICT_START_TOKEN.size(); beg += DICT_START_TOKEN.size();
size_t end = in.find(DICT_END_TOKEN,beg); size_t end = in.find(DICT_END_TOKEN,beg);
if( end == -1 ) if((int) end == -1 )
{ {
return ""; return "";
} }
...@@ -92,7 +92,7 @@ void FilterPredictor::obtainDecodeParams(Object *objectWithStream, std::string & ...@@ -92,7 +92,7 @@ void FilterPredictor::obtainDecodeParams(Object *objectWithStream, std::string &
for(; it != params.end();it++) for(; it != params.end();it++)
{ {
size_t pos = dictStr.find((*it).first); size_t pos = dictStr.find((*it).first);
if( pos != -1 ) if((int) pos != -1 )
{ {
pos += (*it).first.size(); pos += (*it).first.size();
...@@ -123,7 +123,7 @@ void FilterPredictor::initialize(Object *objectWithStream) ...@@ -123,7 +123,7 @@ void FilterPredictor::initialize(Object *objectWithStream)
objectWithStream->getHeader(content); objectWithStream->getHeader(content);
// we need to parse the header of file to obtain the decoder parameter // we need to parse the header of file to obtain the decoder parameter
size_t position = content.find(DECODE_PARAM_TOKEN); size_t position = content.find(DECODE_PARAM_TOKEN);
if( position != -1) if((int) position != -1)
{ {
position += DECODE_PARAM_TOKEN.size(); position += DECODE_PARAM_TOKEN.size();
std::string dictStr = getDictionaryContentStr(content,position); std::string dictStr = getDictionaryContentStr(content,position);
......
...@@ -47,7 +47,7 @@ void FlateDecode::initialize(Object * objectWithStream) ...@@ -47,7 +47,7 @@ void FlateDecode::initialize(Object * objectWithStream)
std::string head; std::string head;
objectWithStream->getHeader(head); objectWithStream->getHeader(head);
if( head.find(FilterPredictor::DECODE_PARAM_TOKEN) != -1 ) if((int) head.find(FilterPredictor::DECODE_PARAM_TOKEN) != -1 )
{ {
_predict = new FilterPredictor(); _predict = new FilterPredictor();
_predict->initialize(objectWithStream); _predict->initialize(objectWithStream);
......
...@@ -59,7 +59,7 @@ void LZWDecode::initialize(Object * objectWithStream) ...@@ -59,7 +59,7 @@ void LZWDecode::initialize(Object * objectWithStream)
std::string head; std::string head;
objectWithStream->getHeader(head); objectWithStream->getHeader(head);
if( head.find(FilterPredictor::DECODE_PARAM_TOKEN) != -1 ) if((int) head.find(FilterPredictor::DECODE_PARAM_TOKEN) != -1 )
{ {
_predict = new FilterPredictor(); _predict = new FilterPredictor();
_predict->initialize(objectWithStream); _predict->initialize(objectWithStream);
......
...@@ -59,7 +59,7 @@ namespace merge_lib ...@@ -59,7 +59,7 @@ namespace merge_lib
} }
bool _wasCropBoxHandlerCalled() bool _wasCropBoxHandlerCalled()
{ {
return (_page->getObjectContent().find("/BBox") != -1) ? true : false; return ((int)_page->getObjectContent().find("/BBox") != -1) ? true : false;
} }
}; };
} }
......
...@@ -369,7 +369,7 @@ bool Object::_findObject(const std::string & token, Object* & foundObject, unsig ...@@ -369,7 +369,7 @@ bool Object::_findObject(const std::string & token, Object* & foundObject, unsig
{ {
_isPassed = true; _isPassed = true;
tokenPositionInContent = Parser::findToken(_content,token); tokenPositionInContent = Parser::findToken(_content,token);
if(tokenPositionInContent != -1) if((int)tokenPositionInContent != -1)
{ {
foundObject = this; foundObject = this;
return true; return true;
...@@ -456,12 +456,12 @@ bool Object::getStream(std::string & stream) ...@@ -456,12 +456,12 @@ bool Object::getStream(std::string & stream)
bool Object::_getStreamFromContent(std::string & stream) bool Object::_getStreamFromContent(std::string & stream)
{ {
size_t stream_begin = _content.find("stream"); size_t stream_begin = _content.find("stream");
if( stream_begin == -1 ) if((int) stream_begin == -1 )
{ {
return false; return false;
} }
size_t stream_end = _content.find("endstream",stream_begin); size_t stream_end = _content.find("endstream",stream_begin);
if( stream_end == -1 ) if((int) stream_end == -1 )
{ {
return false; return false;
} }
...@@ -558,7 +558,7 @@ std::string Object::getNameSimpleValue(const std::string &content, const std::st ...@@ -558,7 +558,7 @@ std::string Object::getNameSimpleValue(const std::string &content, const std::st
Object* Object::findPatternInObjOrParents(const std::string &pattern) Object* Object::findPatternInObjOrParents(const std::string &pattern)
{ {
std::string content=getObjectContent(); std::string content=getObjectContent();
if( Parser::findToken(content,pattern,0) != -1 ) if((int) Parser::findToken(content,pattern,0) != -1 )
{ {
return this; return this;
} }
...@@ -569,7 +569,7 @@ Object* Object::findPatternInObjOrParents(const std::string &pattern) ...@@ -569,7 +569,7 @@ Object* Object::findPatternInObjOrParents(const std::string &pattern)
{ {
unsigned int startOfParent = content.find("/Parent"); unsigned int startOfParent = content.find("/Parent");
unsigned int endOfParent = content.find(" R", startOfParent); unsigned int endOfParent = content.find(" R", startOfParent);
if(startOfParent == -1) if((int)startOfParent == -1)
{ {
break; break;
} }
...@@ -581,7 +581,7 @@ Object* Object::findPatternInObjOrParents(const std::string &pattern) ...@@ -581,7 +581,7 @@ Object* Object::findPatternInObjOrParents(const std::string &pattern)
parent = parents[0]; parent = parents[0];
std::string parentContent = parent->getObjectContent(); std::string parentContent = parent->getObjectContent();
unsigned int startOfPattern = parentContent.find(pattern); unsigned int startOfPattern = parentContent.find(pattern);
if(startOfPattern == -1) if((int)startOfPattern == -1)
{ {
content = parentContent; content = parentContent;
continue; continue;
......
...@@ -167,6 +167,8 @@ void _recalculateAnnotsCoordinates(Object * annotation, ...@@ -167,6 +167,8 @@ void _recalculateAnnotsCoordinates(Object * annotation,
const Rectangle & outputPagesRectangle, const Rectangle & outputPagesRectangle,
const MergePageDescription & description) const MergePageDescription & description)
{ {
Q_UNUSED(outputPagesRectangle);
Q_UNUSED(basePagesRectangle);
std::string annotsRectangleName("/Rect"); std::string annotsRectangleName("/Rect");
Object * objectWithRectangle; Object * objectWithRectangle;
unsigned int fake; unsigned int fake;
...@@ -193,7 +195,7 @@ static void _updateAnnotParentPage(Object *annotation,Object *newParentPage) ...@@ -193,7 +195,7 @@ static void _updateAnnotParentPage(Object *annotation,Object *newParentPage)
std::string &annotContent = annotation->getObjectContent(); std::string &annotContent = annotation->getObjectContent();
size_t startOfP = Parser::findTokenName(annotContent,strP); size_t startOfP = Parser::findTokenName(annotContent,strP);
if( startOfP == -1 ) if((int) startOfP == -1 )
{ {
return; return;
} }
...@@ -229,12 +231,12 @@ static void _updateAnnotParentPage(Object *annotation,Object *newParentPage) ...@@ -229,12 +231,12 @@ static void _updateAnnotParentPage(Object *annotation,Object *newParentPage)
static void _updateAnnotFormColor(Object *annotation ) static void _updateAnnotFormColor(Object *annotation )
{ {
std::string &objectContent = annotation->getObjectContent(); std::string &objectContent = annotation->getObjectContent();
if( objectContent.find("/Widget") == -1 ) if((int) objectContent.find("/Widget") == -1 )
{ {
return; return;
} }
size_t startOfAP = Parser::findTokenName(objectContent,"/AP"); size_t startOfAP = Parser::findTokenName(objectContent,"/AP");
if( startOfAP == -1 ) if((int) startOfAP == -1 )
{ {
return; return;
} }
...@@ -262,7 +264,7 @@ static void _updateAnnotFormColor(Object *annotation ) ...@@ -262,7 +264,7 @@ static void _updateAnnotFormColor(Object *annotation )
{ {
if( token == "f" || token == "F" ) if( token == "f" || token == "F" )
{ {
if( found != -1 ) if((int) found != -1 )
{ {
decodedStream[found] = ' '; decodedStream[found] = ' ';
} }
...@@ -272,7 +274,7 @@ static void _updateAnnotFormColor(Object *annotation ) ...@@ -272,7 +274,7 @@ static void _updateAnnotFormColor(Object *annotation )
// Then we need to update Filter section (if any) // Then we need to update Filter section (if any)
std::string filterStr = "/Filter"; std::string filterStr = "/Filter";
size_t startOfFlate = Parser::findTokenName(content,filterStr); size_t startOfFlate = Parser::findTokenName(content,filterStr);
if( startOfFlate != -1 ) if((int) startOfFlate != -1 )
{ {
size_t endOfFlate = Parser::findEndOfElementContent(content,startOfFlate+filterStr.size()); size_t endOfFlate = Parser::findEndOfElementContent(content,startOfFlate+filterStr.size());
childWithAP->eraseContent(startOfFlate,endOfFlate-startOfFlate); childWithAP->eraseContent(startOfFlate,endOfFlate-startOfFlate);
...@@ -285,7 +287,7 @@ static void _updateAnnotFormColor(Object *annotation ) ...@@ -285,7 +287,7 @@ static void _updateAnnotFormColor(Object *annotation )
// update the length field // update the length field
std::string lengthStr = "/Length"; std::string lengthStr = "/Length";
size_t startOfLength = Parser::findTokenName(content,lengthStr,0); size_t startOfLength = Parser::findTokenName(content,lengthStr,0);
if( startOfLength != -1 ) if((int) startOfLength != -1 )
{ {
size_t endOfLength = Parser::findEndOfElementContent(content,startOfLength + lengthStr.size()); size_t endOfLength = Parser::findEndOfElementContent(content,startOfLength + lengthStr.size());
childWithAP->eraseContent(startOfLength,endOfLength-startOfLength); childWithAP->eraseContent(startOfLength,endOfLength-startOfLength);
...@@ -296,10 +298,10 @@ static void _updateAnnotFormColor(Object *annotation ) ...@@ -296,10 +298,10 @@ static void _updateAnnotFormColor(Object *annotation )
// update the stream of object with new content // update the stream of object with new content
std::string stream("stream"); std::string stream("stream");
size_t leftBoundOfContentStream = content.find(stream); size_t leftBoundOfContentStream = content.find(stream);
if( leftBoundOfContentStream != -1 ) if((int) leftBoundOfContentStream != -1 )
{ {
size_t rightBoundOfContentStream = content.find("endstream", leftBoundOfContentStream); size_t rightBoundOfContentStream = content.find("endstream", leftBoundOfContentStream);
if( rightBoundOfContentStream == -1 ) if((int) rightBoundOfContentStream == -1 )
{ {
rightBoundOfContentStream = content.size() - 1; rightBoundOfContentStream = content.size() - 1;
} }
...@@ -323,7 +325,7 @@ static void processBasePageResources(Object *basePage) ...@@ -323,7 +325,7 @@ static void processBasePageResources(Object *basePage)
return; return;
} }
std::string resourceToken = "/Resources"; std::string resourceToken = "/Resources";
if( Parser::findTokenName(basePage->getObjectContent(),resourceToken) == -1 ) if((int) Parser::findTokenName(basePage->getObjectContent(),resourceToken) == -1 )
{ {
// it seems base page does not have resources, they can be located in parent! // it seems base page does not have resources, they can be located in parent!
Object *resource = basePage->findPatternInObjOrParents(resourceToken); Object *resource = basePage->findPatternInObjOrParents(resourceToken);
...@@ -331,20 +333,20 @@ static void processBasePageResources(Object *basePage) ...@@ -331,20 +333,20 @@ static void processBasePageResources(Object *basePage)
{ {
std::string &resContStr = resource->getObjectContent(); std::string &resContStr = resource->getObjectContent();
size_t startOfRes = Parser::findTokenName(resContStr,resourceToken); size_t startOfRes = Parser::findTokenName(resContStr,resourceToken);
if( startOfRes == -1 ) if((int) startOfRes == -1 )
{ {
// no resources at all // no resources at all
return; return;
} }
size_t endOfRes = Parser::findEndOfElementContent(resContStr,startOfRes + resourceToken.size()); size_t endOfRes = Parser::findEndOfElementContent(resContStr,startOfRes + resourceToken.size());
if( endOfRes == -1 ) if((int) endOfRes == -1 )
{ {
return; // broken resources return; // broken resources
} }
std::string resourceContent = resContStr.substr(startOfRes,endOfRes-startOfRes); std::string resourceContent = resContStr.substr(startOfRes,endOfRes-startOfRes);
size_t positionToInsert = basePage->getObjectContent().find("<<"); size_t positionToInsert = basePage->getObjectContent().find("<<");
if( positionToInsert == -1 ) if((int) positionToInsert == -1 )
{ {
positionToInsert = 0; positionToInsert = 0;
resourceContent.insert(0,"<<"); resourceContent.insert(0,"<<");
...@@ -479,7 +481,7 @@ void Page::merge(Page * sourcePage, Document * parentDocument, MergePageDescript ...@@ -479,7 +481,7 @@ void Page::merge(Page * sourcePage, Document * parentDocument, MergePageDescript
rotationHandler.processObjectContent(); rotationHandler.processObjectContent();
description.basePageTransformation.addRotation(_rotation); description.basePageTransformation.addRotation(_rotation);
if( sourcePage->_root->getObjectContent().find("/Annots") != -1 ) if((int) sourcePage->_root->getObjectContent().find("/Annots") != -1 )
{ {
Object *crop = sourcePage->_root->findPatternInObjOrParents("/CropBox"); Object *crop = sourcePage->_root->findPatternInObjOrParents("/CropBox");
if( crop ) if( crop )
......
...@@ -73,10 +73,10 @@ unsigned int PageElementHandler::_findEndOfElementContent(unsigned int startOfPa ...@@ -73,10 +73,10 @@ unsigned int PageElementHandler::_findEndOfElementContent(unsigned int startOfPa
static std::string whitespacesAndDelimeters(" \t\f\v\n\r<<[/"); static std::string whitespacesAndDelimeters(" \t\f\v\n\r<<[/");
unsigned int foundSlash = _pageContent.find("/", startOfPageElement + 1); unsigned int foundSlash = _pageContent.find("/", startOfPageElement + 1);
std::string fieldType; std::string fieldType;
while(foundSlash != -1) while((int)foundSlash != -1)
{ {
unsigned int foundWhitespace = _pageContent.find_first_of(whitespacesAndDelimeters, foundSlash + 1); unsigned int foundWhitespace = _pageContent.find_first_of(whitespacesAndDelimeters, foundSlash + 1);
if(foundWhitespace != -1) if((int)foundWhitespace != -1)
fieldType = _pageContent.substr(foundSlash + 1, foundWhitespace - foundSlash - 1); fieldType = _pageContent.substr(foundSlash + 1, foundWhitespace - foundSlash - 1);
else else
break; break;
......
...@@ -46,7 +46,7 @@ namespace merge_lib ...@@ -46,7 +46,7 @@ namespace merge_lib
void processObjectContent() void processObjectContent()
{ {
unsigned int startOfPageElement = _findStartOfPageElement(); unsigned int startOfPageElement = _findStartOfPageElement();
if(startOfPageElement != -1) if((int)startOfPageElement != -1)
_processObjectContent(startOfPageElement); _processObjectContent(startOfPageElement);
if(_nextHandler) if(_nextHandler)
_nextHandler->processObjectContent(); _nextHandler->processObjectContent();
...@@ -55,7 +55,7 @@ namespace merge_lib ...@@ -55,7 +55,7 @@ namespace merge_lib
void changeObjectContent() void changeObjectContent()
{ {
unsigned int startOfPageElement = _findStartOfPageElement(); unsigned int startOfPageElement = _findStartOfPageElement();
if(startOfPageElement != -1) if((int)startOfPageElement != -1)
_changeObjectContent(startOfPageElement); _changeObjectContent(startOfPageElement);
else else
_pageElementNotFound(); _pageElementNotFound();
......
This diff is collapsed.
...@@ -42,14 +42,14 @@ _rectangleName(rectangleName) ...@@ -42,14 +42,14 @@ _rectangleName(rectangleName)
{ {
unsigned int rectanglePosition = Parser::findToken(content,rectangleName); unsigned int rectanglePosition = Parser::findToken(content,rectangleName);
if( rectanglePosition == -1 ) if((int) rectanglePosition == -1 )
{ {
std::cerr<<"Unable to find rectangle name "<<rectangleName<<" in content\n"; std::cerr<<"Unable to find rectangle name "<<rectangleName<<" in content\n";
} }
size_t beg = content.find("[",rectanglePosition); size_t beg = content.find("[",rectanglePosition);
size_t end = content.find("]",rectanglePosition); size_t end = content.find("]",rectanglePosition);
if( beg != -1 && end != -1 ) if((int) beg != -1 && (int) end != -1 )
{ {
std::string arr = content.substr(beg+1,end-beg-1); std::string arr = content.substr(beg+1,end-beg-1);
std::stringstream in; std::stringstream in;
...@@ -119,7 +119,7 @@ void Rectangle::updateRectangle(Object * objectWithRectangle, const char * delim ...@@ -119,7 +119,7 @@ void Rectangle::updateRectangle(Object * objectWithRectangle, const char * delim
std::string objectContent = objectWithMatrix->getObjectContent(); std::string objectContent = objectWithMatrix->getObjectContent();
unsigned int matrixPosition = Parser::findToken(objectContent,"/Matrix"); unsigned int matrixPosition = Parser::findToken(objectContent,"/Matrix");
if(matrixPosition == -1) if((int)matrixPosition == -1)
continue; continue;
unsigned int matrixValueLeftBound = objectContent.find("[", matrixPosition); unsigned int matrixValueLeftBound = objectContent.find("[", matrixPosition);
unsigned int matrixValueRightBound = objectContent.find("]", matrixValueLeftBound) + 1; unsigned int matrixValueRightBound = objectContent.find("]", matrixValueLeftBound) + 1;
......
...@@ -47,8 +47,8 @@ class UBWebPluginFactory : public QWebPluginFactory ...@@ -47,8 +47,8 @@ class UBWebPluginFactory : public QWebPluginFactory
QObject* create(const QString &mimeType, const QUrl &url, const QStringList &argumentNames, const QStringList &argumentValues) const QObject* create(const QString &mimeType, const QUrl &url, const QStringList &argumentNames, const QStringList &argumentValues) const
{ {
Q_UNUSED(url); Q_UNUSED(url);
Q_UNUSED(argumentNames); Q_UNUSED(argumentNames);
Q_UNUSED(argumentValues); Q_UNUSED(argumentValues);
if (mimeType == "application/x-ub-pdf") if (mimeType == "application/x-ub-pdf")
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment