Copilot commented on code in PR #3092:
URL: https://github.com/apache/tika/pull/3092#discussion_r3886824982
##########
tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-cad-module/src/test/java/org/apache/tika/parser/dwg/DWGParserTest.java:
##########
@@ -276,16 +276,17 @@ public void testDWGReadexe() throws Exception {
assertNotNull(root.get(DWG.APPLICATION_COMMENT));
assertContains("AutoCAD", root.get(DWG.PRODUCT_INFO));
- // Thumbnail embedded as INLINE
+ // the THUMBNAILIMAGE section is the drawing's THUMBNAIL embedded
document
Review Comment:
For consistency with nearby comments in this test, capitalize the start of
this sentence.
##########
tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-cad-module/src/main/java/org/apache/tika/parser/dwg/DWGReadParser.java:
##########
@@ -416,8 +416,9 @@ private void parseThumbnail(JsonParser jsonParser,
XHTMLContentHandler xhtml,
EmbeddedDocumentUtil.getEmbeddedDocumentExtractor(context);
Metadata embeddedMetadata = new Metadata();
embeddedMetadata.set(TikaCoreProperties.RESOURCE_NAME_KEY,
"thumbnail");
+ //the drawing's preview image, not a picture placed in the drawing
Review Comment:
Comment style is inconsistent with the rest of the file: there should be a
space after `//` and the sentence should start with a capital letter.
--
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]
