kleb-prog commented on code in PR #496:
URL: https://github.com/apache/poi/pull/496#discussion_r1285107535


##########
poi/src/main/java/org/apache/poi/ss/formula/FormulaParser.java:
##########
@@ -1182,8 +1182,6 @@ private SheetIdentifier parseSheetName(boolean 
isSndPartOfQuotedRange) {
                         if (look == '\''){
                             // Any single quotes which were already present in 
the sheet name will be converted to double single quotes ('')
                             // so switch back to single quote
-                            sb.appendCodePoint(look);
-                            nextChar();

Review Comment:
   It's not necessary to do it here because on this step variable "look" has 
second single quote of the pair of single quotes and will be appended to string 
builder on next iteration of the loop



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to