From 9ad6fb42d6e36545cb073a01a13ee1154ed36c47 Mon Sep 17 00:00:00 2001 From: w3stling Date: Thu, 10 Aug 2023 20:27:00 +0200 Subject: [PATCH] Fixed code smell --- .../apptasticsoftware/rssreader/AbstractRssReader.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/apptasticsoftware/rssreader/AbstractRssReader.java b/src/main/java/com/apptasticsoftware/rssreader/AbstractRssReader.java index c41f6d9..3c463f1 100644 --- a/src/main/java/com/apptasticsoftware/rssreader/AbstractRssReader.java +++ b/src/main/java/com/apptasticsoftware/rssreader/AbstractRssReader.java @@ -458,7 +458,7 @@ else if (firstChar == 13 || Character.isWhitespace(firstChar)) { class RssItemIterator implements Iterator { private final StringBuilder textBuilder; private final InputStream is; - private final Stack elementStack; + private final Deque elementStack; private XMLStreamReader reader; private C channel; private I item = null; @@ -471,7 +471,7 @@ public RssItemIterator(InputStream is) { this.is = is; nextItem = null; textBuilder = new StringBuilder(); - elementStack = new Stack<>(); + elementStack = new ArrayDeque<>(); try { var xmlInFact = XMLInputFactory.newInstance(); @@ -564,7 +564,7 @@ void parseStartElement() { elementName = reader.getLocalName(); var prefix = reader.getPrefix(); var nsLocalName = toNsName(prefix, elementName); - elementStack.push(nsLocalName); + elementStack.addLast(nsLocalName); if ("channel".equals(nsLocalName) || "feed".equals(nsLocalName)) { channel = createChannel(); @@ -625,7 +625,7 @@ boolean parseEndElement() { var nsLocalName = toNsName(prefix, localName); var text = textBuilder.toString().trim(); var elementFullPath = getElementFullPath(); - elementStack.pop(); + elementStack.removeLast(); if (isChannelPart) parseChannelCharacters(channel, prefix, elementName, elementFullPath, text);