kinow commented on a change in pull request #9: Imaging-163
URL: https://github.com/apache/commons-imaging/pull/9#discussion_r339462304
##########
File path: src/main/java/org/apache/commons/imaging/ImageParser.java
##########
@@ -682,8 +688,10 @@ public abstract Dimension getImageSize(ByteSource
byteSource, Map<String, Object
* parser implementation.
* @throws IOException In the event of unsuccessful read or access
operation.
*/
- public abstract String getXmpXml(ByteSource byteSource, Map<String,
Object> params)
- throws ImageReadException, IOException;
+ public String getXmpXml(final ByteSource byteSource, final Map<String,
Object> params)
Review comment:
Reading the issue, and this last comment, I agree about the issue being
actually a bad design. IMO, we should move that method to an interface, perhaps
``. Then implementations can choose to implement it when appropriate.
So for `PcxImageParser` it would simply not implement it.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
