Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss
Replace POMReader in maven-repo-handler with DOM based reader
2 views
Skip to first unread message
Thomas Koch
Oct 27, 2012, 12:10:03 PM10/27/12
to
Hi,
the current POMReader class in maven-repo-handler is based on a streaming XML
reader API. The read function has over 200 lines with 7+ levels of nesting and
30+ local variables. The reader also intermixes XML parsing and maven POM
processing logic.
I'm thinking about replacing the current POMReader with a DOM based class.
This would consume more memory and be slower, but neither should be a problem
here. The new class would wrap the DOM object and have getters to extract data
structures that interest us: Dependencies, Plugins, ...
Regards,
Thomas Koch, http://www.koch.ro
--
To UNSUBSCRIBE, email to debian-ja...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listm...@lists.debian.org
Archive: http://lists.debian.org/201210271806...@koch.ro
the current POMReader class in maven-repo-handler is based on a streaming XML
reader API. The read function has over 200 lines with 7+ levels of nesting and
30+ local variables. The reader also intermixes XML parsing and maven POM
processing logic.
I'm thinking about replacing the current POMReader with a DOM based class.
This would consume more memory and be slower, but neither should be a problem
here. The new class would wrap the DOM object and have getters to extract data
structures that interest us: Dependencies, Plugins, ...
Regards,
Thomas Koch, http://www.koch.ro
--
To UNSUBSCRIBE, email to debian-ja...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listm...@lists.debian.org
Archive: http://lists.debian.org/201210271806...@koch.ro
0 new messages