Re: [IMAGING] Pull request

2018-11-30 Thread Gilles
Hi. On Fri, 30 Nov 2018 14:00:52 +, Sukant Kumar wrote: Hello Everyone, I am Shukant Pal, look at my PR for Commons Imaging https://github.com/apache/commons-imaging/pull/38. I want to get it merged, how to do that? A committer has to do it. IIRC the main committer of [Imaging] (Bruno) is

[IMAGING] Pull request

2018-11-30 Thread Sukant Kumar
Hello Everyone, I am Shukant Pal, look at my PR for Commons Imaging https://github.com/apache/commons-imaging/pull/38. I want to get it merged, how to do that? Regards, ShukantPal

[GitHub] commons-imaging pull request #38: Cleaned up PNG component.

2018-11-29 Thread SukantPal
GitHub user SukantPal opened a pull request: https://github.com/apache/commons-imaging/pull/38 Cleaned up PNG component. Following changes were made (overview): PngWriter.ImageHeader removed (replaced by making PngChunkIhdr mutable); divided PngWriter.writeImage (by creating pixel

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-imaging/pull/35 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-16 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210549560 --- Diff: src/main/java/org/apache/commons/imaging/formats/bmp/BmpImageParser.java --- @@ -327,7 +331,7 @@ private BmpImageContents readImageContents(f

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-16 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210546814 --- Diff: src/main/java/org/apache/commons/imaging/formats/bmp/BmpImageParser.java --- @@ -327,7 +331,7 @@ private BmpImageContents readImageContents(f

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-16 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210524716 --- Diff: src/main/java/org/apache/commons/imaging/icc/IccTag.java --- @@ -73,11 +76,12 @@ private IccTagDataType getIccTagDataType(final int quad) {

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-16 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210523866 --- Diff: src/main/java/org/apache/commons/imaging/formats/psd/PsdHeaderInfo.java --- @@ -45,9 +50,14 @@ public PsdHeaderInfo(final int version, final

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-16 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210521304 --- Diff: src/main/java/org/apache/commons/imaging/formats/bmp/BmpImageParser.java --- @@ -327,7 +331,7 @@ private BmpImageContents readImageContents(f

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-16 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210519489 --- Diff: src/main/java/org/apache/commons/imaging/formats/png/chunks/PngChunkText.java --- @@ -42,9 +47,9 @@ public PngChunkText(final int length, fi

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-15 Thread sebbASF
Github user sebbASF commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210327438 --- Diff: src/main/java/org/apache/commons/imaging/formats/bmp/BmpImageParser.java --- @@ -327,7 +331,7 @@ private BmpImageContents readImageContents

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-15 Thread garydgregory
Github user garydgregory commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210287763 --- Diff: src/test/java/org/apache/commons/imaging/formats/jpeg/exif/WriteExifMetadataExampleTest.java --- @@ -27,7 +27,7 @@ import org.ap

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-15 Thread garydgregory
Github user garydgregory commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210287591 --- Diff: src/main/java/org/apache/commons/imaging/icc/IccTag.java --- @@ -73,11 +76,12 @@ private IccTagDataType getIccTagDataType(final int qu

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-15 Thread garydgregory
Github user garydgregory commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210287298 --- Diff: src/main/java/org/apache/commons/imaging/formats/tiff/TiffField.java --- @@ -355,9 +360,14 @@ private String getValueDescription(final

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-15 Thread garydgregory
Github user garydgregory commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210287176 --- Diff: src/main/java/org/apache/commons/imaging/formats/psd/PsdImageContents.java --- @@ -40,9 +45,14 @@ public PsdImageContents(final PsdHea

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-15 Thread garydgregory
Github user garydgregory commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210287038 --- Diff: src/main/java/org/apache/commons/imaging/formats/psd/PsdHeaderInfo.java --- @@ -45,9 +50,14 @@ public PsdHeaderInfo(final int version,

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-15 Thread garydgregory
Github user garydgregory commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210286764 --- Diff: src/main/java/org/apache/commons/imaging/formats/png/chunks/PngChunkText.java --- @@ -42,9 +47,9 @@ public PngChunkText(final int len

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-08-15 Thread garydgregory
Github user garydgregory commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/35#discussion_r210286324 --- Diff: src/main/java/org/apache/commons/imaging/formats/bmp/BmpImageParser.java --- @@ -327,7 +331,7 @@ private BmpImageContents readImageCon

[GitHub] commons-imaging pull request #36: Avoid jpeg parse fail due to jfxx and exif

2018-07-21 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/36#discussion_r204226858 --- Diff: src/main/java/org/apache/commons/imaging/formats/jpeg/JpegImageParser.java --- @@ -312,7 +312,12 @@ private void dumpSegments(final List v) {

[GitHub] commons-imaging pull request #34: IMAGING-188

2018-07-21 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/34#discussion_r204226800 --- Diff: pom.xml --- @@ -209,6 +209,12 @@ 2.6 test + + + com.github.jai-imageio + jai

[GitHub] commons-imaging pull request #37: add size guard against corrupted icns file...

2018-04-20 Thread gregsh
GitHub user gregsh opened a pull request: https://github.com/apache/commons-imaging/pull/37 add size guard against corrupted icns files The problematic incs file: groovy-1.7.5/bin/groovy.icns From https://bintray.com/groovy/maven/groovy/1.7.5#files You can merge this

[GitHub] commons-imaging pull request #36: Avoid jpeg parse fail due to jfxx and exif

2018-03-28 Thread hyunuck
GitHub user hyunuck opened a pull request: https://github.com/apache/commons-imaging/pull/36 Avoid jpeg parse fail due to jfxx and exif - add jfxx signature when parsing jpeg - Avoid jpeg parse fail due to exif You can merge this pull request into a Git repository by running:

[GitHub] commons-imaging pull request #35: IMAGING-154 Removed Debug class

2018-02-05 Thread kinow
GitHub user kinow opened a pull request: https://github.com/apache/commons-imaging/pull/35 IMAGING-154 Removed Debug class Removed the `Debug` and `ImageDump` classes. Also removed methods with the `verbose` parameter, as well as any `System.out.println`'s. Tried to keep the change

[GitHub] commons-imaging pull request #15: Imaging-136

2017-12-31 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/15#discussion_r159147397 --- Diff: src/main/java/org/apache/commons/imaging/formats/jpeg/decoder/JpegDecoder.java --- @@ -38,7 +41,7 @@ import static org.apache.com

[GitHub] commons-imaging pull request #15: Imaging-136

2017-12-31 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/15#discussion_r159147391 --- Diff: src/main/java/org/apache/commons/imaging/common/BinaryFunctions.java --- @@ -317,4 +317,23 @@ public static void copyStreamToStream(final In

[GitHub] commons-imaging pull request #15: Imaging-136

2017-12-31 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/15#discussion_r159147410 --- Diff: src/test/data/specificTests/info.txt --- @@ -0,0 +1 @@ +Licensed to the Apache Software Foundation (ASF) under one or more contributor lic

[GitHub] commons-imaging pull request #15: Imaging-136

2017-12-31 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/15#discussion_r159147413 --- Diff: src/test/java/org/apache/commons/imaging/formats/jpeg/specific/JpegImageParserTest.java --- @@ -0,0 +1,48 @@ +/* + * Copyright 2015

[GitHub] commons-imaging pull request #15: Imaging-136

2017-12-31 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/15#discussion_r159147395 --- Diff: src/main/java/org/apache/commons/imaging/formats/jpeg/JpegConstants.java --- @@ -125,7 +136,9 @@ SOF5_MARKER, SOF6_M

[GitHub] commons-imaging pull request #32: add test for DataReaderStripsTest::applyPr...

2017-12-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-imaging/pull/32 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-imaging pull request #33: Imaging-209 PnmImageParser throws ImageWri...

2017-12-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-imaging/pull/33 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-imaging pull request #31: add-missing-license-headers

2017-12-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-imaging/pull/31 --- - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org

[GitHub] commons-imaging pull request #34: IMAGING-188

2017-12-07 Thread DasBueroAmDraht
GitHub user DasBueroAmDraht opened a pull request: https://github.com/apache/commons-imaging/pull/34 IMAGING-188 https://issues.apache.org/jira/browse/IMAGING-188 commits: - support for all icns types listed on wikipedia: https://en.wikipedia.org/wiki/Apple_Icon_Image_

[GitHub] commons-imaging pull request #33: Imaging-209 PnmImageParser throws ImageWri...

2017-12-04 Thread aesy
GitHub user aesy opened a pull request: https://github.com/apache/commons-imaging/pull/33 Imaging-209 PnmImageParser throws ImageWriteException when writing if PnmImageParser.PARAM_KEY_PNM_RAWBITS is provided. I fixed the issue and added a test to cover it. Issue ca

[GitHub] commons-imaging pull request #32: add test for DataReaderStripsTest::applyPr...

2017-10-14 Thread testingsavvy
GitHub user testingsavvy opened a pull request: https://github.com/apache/commons-imaging/pull/32 add test for DataReaderStripsTest::applyPredictor This PR adds a new test for `DataReaderStripsTest::applyPredictor`, so that it can be tested in isolation. You can merge this pull req

[GitHub] commons-imaging pull request #30: add-tests

2017-09-19 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r139785201 --- Diff: src/test/java/org/apache/commons/imaging/formats/jpeg/xmp/JpegXmpParserTest.java --- @@ -1,3 +1,19 @@ +/* --- End diff --

[GitHub] commons-imaging pull request #31: add-missing-license-headers

2017-09-19 Thread TheRealHaui
GitHub user TheRealHaui opened a pull request: https://github.com/apache/commons-imaging/pull/31 add-missing-license-headers Added missing license headers. You can merge this pull request into a Git repository by running: $ git pull https://github.com/TheRealHaui/commons-imagin

[GitHub] commons-imaging pull request #30: add-tests

2017-08-09 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r132311416 --- Diff: pom.xml --- @@ -209,6 +209,12 @@ 2.5 test + --- End diff -- Understand your point.

[GitHub] commons-imaging pull request #30: add-tests

2017-08-09 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r132310660 --- Diff: pom.xml --- @@ -209,6 +209,12 @@ 2.5 test + --- End diff -- Indeed, and I may even use it

[GitHub] commons-imaging pull request #30: add-tests

2017-08-09 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r132308184 --- Diff: pom.xml --- @@ -209,6 +209,12 @@ 2.5 test + --- End diff -- Well, Equalsverifier ch

[GitHub] commons-imaging pull request #30: add-tests

2017-08-09 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r132307800 --- Diff: src/test/java/org/apache/commons/imaging/formats/tiff/taginfos/TagInfoByteOrShortTest.java --- @@ -0,0 +1,55 @@ +/* + * Licens

[GitHub] commons-imaging pull request #30: add-tests

2017-08-09 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r132306981 --- Diff: src/test/java/org/apache/commons/imaging/formats/pnm/PgmFileInfoTest.java --- @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache S

[GitHub] commons-imaging pull request #30: add-tests

2017-08-09 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r132306138 --- Diff: src/test/java/org/apache/commons/imaging/formats/pnm/PgmFileInfoTest.java --- @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache S

[GitHub] commons-imaging pull request #18: IMAGING-168 installing package with Swedis...

2017-08-06 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/18#discussion_r131540224 --- Diff: src/main/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcParser.java --- @@ -248,6 +258,35 @@ public PhotoshopApp13Data parsePhotoshopS

[GitHub] commons-imaging pull request #18: IMAGING-168 installing package with Swedis...

2017-08-06 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/18#discussion_r131540245 --- Diff: src/main/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcRecord.java --- @@ -50,6 +50,10 @@ public IptcRecord(final IptcType iptcType,

[GitHub] commons-imaging pull request #30: add-tests

2017-08-06 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r131538892 --- Diff: src/test/java/org/apache/commons/imaging/formats/pnm/PgmFileInfoTest.java --- @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache Softwar

[GitHub] commons-imaging pull request #30: add-tests

2017-08-06 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r131538887 --- Diff: src/test/java/org/apache/commons/imaging/formats/pnm/PgmFileInfoTest.java --- @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache Softwar

[GitHub] commons-imaging pull request #30: add-tests

2017-08-06 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r131538792 --- Diff: pom.xml --- @@ -209,6 +209,12 @@ 2.5 test + --- End diff -- -0 Looks like it reduces the

[GitHub] commons-imaging pull request #30: add-tests

2017-08-06 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r131538871 --- Diff: src/test/java/org/apache/commons/imaging/formats/jpeg/xmp/JpegXmpParserTest.java --- @@ -1,3 +1,19 @@ +/* --- End diff -- G

[GitHub] commons-imaging pull request #30: add-tests

2017-08-06 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/30#discussion_r131538915 --- Diff: src/test/java/org/apache/commons/imaging/formats/tiff/taginfos/TagInfoByteOrShortTest.java --- @@ -0,0 +1,55 @@ +/* + * Licensed to

[GitHub] commons-imaging pull request #30: add-tests

2017-07-31 Thread TheRealHaui
GitHub user TheRealHaui opened a pull request: https://github.com/apache/commons-imaging/pull/30 add-tests Added new Unit Tests. You can merge this pull request into a Git repository by running: $ git pull https://github.com/TheRealHaui/commons-imaging add-tests Alternatively

[GitHub] commons-imaging pull request #29: Improve various unit tests style

2017-07-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-imaging/pull/29 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] commons-imaging pull request #29: Improve various unit tests style

2017-07-19 Thread TheRealHaui
GitHub user TheRealHaui opened a pull request: https://github.com/apache/commons-imaging/pull/29 Improve various unit tests style Code formatting changed as desired in #27 and #28. You can merge this pull request into a Git repository by running: $ git pull https://github.com/T

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-19 Thread TheRealHaui
Github user TheRealHaui closed the pull request at: https://github.com/apache/commons-imaging/pull/28 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fe

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127843760 --- Diff: src/test/java/org/apache/commons/imaging/formats/tiff/fieldtypes/FieldTypeByteTest.java --- @@ -0,0 +1,37 @@ +package org.apache.c

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127780127 --- Diff: src/test/java/org/apache/commons/imaging/ImageDumpTest.java --- @@ -0,0 +1,44 @@ +package org.apache.commons.imaging; + +import

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127782716 --- Diff: src/test/java/org/apache/commons/imaging/common/bytesource/ByteSourceTest.java --- @@ -17,24 +17,22 @@ package org.apache.comm

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127781644 --- Diff: src/test/java/org/apache/commons/imaging/ImageDumpTest.java --- @@ -0,0 +1,44 @@ +package org.apache.commons.imaging; + +import

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127780455 --- Diff: src/test/java/org/apache/commons/imaging/ImageDumpTest.java --- @@ -0,0 +1,44 @@ +package org.apache.commons.imaging; + +import

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127782811 --- Diff: src/test/java/org/apache/commons/imaging/common/bytesource/ByteSourceTest.java --- @@ -66,6 +64,22 @@ protected File createTempFile(final

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127782170 --- Diff: src/test/java/org/apache/commons/imaging/common/RgbBufferedImageFactoryTest.java --- @@ -0,0 +1,36 @@ +package org.apache.commons.ima

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127783301 --- Diff: src/test/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcTypeLookupTest.java --- @@ -0,0 +1,28 @@ +package org.apache.commons.i

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127782622 --- Diff: src/test/java/org/apache/commons/imaging/common/bytesource/ByteSourceTest.java --- @@ -17,24 +17,22 @@ package org.apache.comm

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127781095 --- Diff: src/test/java/org/apache/commons/imaging/ImageDumpTest.java --- @@ -0,0 +1,44 @@ +package org.apache.commons.imaging; + +import

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127781035 --- Diff: src/test/java/org/apache/commons/imaging/ImageDumpTest.java --- @@ -0,0 +1,44 @@ +package org.apache.commons.imaging; + +import

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127783226 --- Diff: src/test/java/org/apache/commons/imaging/formats/bmp/BmpWriterRgbTest.java --- @@ -0,0 +1,35 @@ +package org.apache.commons.imaging.fo

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127781676 --- Diff: src/test/java/org/apache/commons/imaging/common/RgbBufferedImageFactoryTest.java --- @@ -0,0 +1,36 @@ +package org.apache.commons.ima

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127782199 --- Diff: src/test/java/org/apache/commons/imaging/common/RgbBufferedImageFactoryTest.java --- @@ -0,0 +1,36 @@ +package org.apache.commons.ima

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127781398 --- Diff: src/test/java/org/apache/commons/imaging/ImageDumpTest.java --- @@ -0,0 +1,44 @@ +package org.apache.commons.imaging; + +import

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127782141 --- Diff: src/test/java/org/apache/commons/imaging/common/RgbBufferedImageFactoryTest.java --- @@ -0,0 +1,36 @@ +package org.apache.commons.ima

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/28#discussion_r127781871 --- Diff: src/test/java/org/apache/commons/imaging/common/RgbBufferedImageFactoryTest.java --- @@ -0,0 +1,36 @@ +package org.apache.commons.ima

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread jbduncan
Github user jbduncan commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127778422 --- Diff: src/test/java/org/apache/commons/imaging/common/bytesource/ByteSourceTest.java --- @@ -66,6 +64,22 @@ protected File createTempFile(final

[GitHub] commons-imaging pull request #28: Increase code coverage one

2017-07-17 Thread TheRealHaui
GitHub user TheRealHaui opened a pull request: https://github.com/apache/commons-imaging/pull/28 Increase code coverage one Added various improvements to selected Unit Tests. You can merge this pull request into a Git repository by running: $ git pull https://github.com/TheReal

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127775941 --- Diff: src/test/java/org/apache/commons/imaging/formats/tiff/fieldtypes/FieldTypeByteTest.java --- @@ -0,0 +1,37 @@ +package org.apache.c

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127770670 --- Diff: src/test/java/org/apache/commons/imaging/formats/pnm/PbmFileInfoTest.java --- @@ -0,0 +1,41 @@ +package org.apache.commons.imaging.

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127770520 --- Diff: src/test/java/org/apache/commons/imaging/formats/pnm/PbmFileInfoTest.java --- @@ -0,0 +1,41 @@ +package org.apache.commons.imaging.

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127769590 --- Diff: src/test/java/org/apache/commons/imaging/formats/jpeg/segments/JfifSegmentTest.java --- @@ -0,0 +1,35 @@ +package org.apache.commo

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127768831 --- Diff: src/test/java/org/apache/commons/imaging/common/bytesource/ByteSourceTest.java --- @@ -66,6 +64,22 @@ protected File createTempFile(fi

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127768274 --- Diff: src/test/java/org/apache/commons/imaging/formats/bmp/BmpWriterRgbTest.java --- @@ -0,0 +1,35 @@ +package org.apache.commons.imaging

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-17 Thread TheRealHaui
Github user TheRealHaui commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127761190 --- Diff: src/test/java/org/apache/commons/imaging/formats/bmp/BmpWriterRgbTest.java --- @@ -0,0 +1,35 @@ +package org.apache.commons.imaging

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-16 Thread onealj
Github user onealj commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127632776 --- Diff: src/test/java/org/apache/commons/imaging/formats/jpeg/segments/JfifSegmentTest.java --- @@ -0,0 +1,35 @@ +package org.apache.commons.im

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-16 Thread onealj
Github user onealj commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127633319 --- Diff: src/test/java/org/apache/commons/imaging/formats/tiff/fieldtypes/FieldTypeByteTest.java --- @@ -0,0 +1,37 @@ +package org.apache.common

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-16 Thread onealj
Github user onealj commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127633025 --- Diff: src/test/java/org/apache/commons/imaging/formats/pnm/PbmFileInfoTest.java --- @@ -0,0 +1,41 @@ +package org.apache.commons.imaging.forma

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-16 Thread onealj
Github user onealj commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127632145 --- Diff: src/test/java/org/apache/commons/imaging/formats/bmp/BmpWriterRgbTest.java --- @@ -0,0 +1,35 @@ +package org.apache.commons.imaging.form

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-16 Thread onealj
Github user onealj commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127632387 --- Diff: src/test/java/org/apache/commons/imaging/formats/bmp/BmpWriterRgbTest.java --- @@ -0,0 +1,35 @@ +package org.apache.commons.imaging.form

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-16 Thread onealj
Github user onealj commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127632531 --- Diff: src/test/java/org/apache/commons/imaging/common/bytesource/ByteSourceTest.java --- @@ -66,6 +64,22 @@ protected File createTempFile(final b

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-16 Thread onealj
Github user onealj commented on a diff in the pull request: https://github.com/apache/commons-imaging/pull/27#discussion_r127632941 --- Diff: src/test/java/org/apache/commons/imaging/formats/pnm/PbmFileInfoTest.java --- @@ -0,0 +1,41 @@ +package org.apache.commons.imaging.forma

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-imaging/pull/27 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] commons-imaging pull request #27: Increase code coverage one

2017-07-13 Thread TheRealHaui
GitHub user TheRealHaui opened a pull request: https://github.com/apache/commons-imaging/pull/27 Increase code coverage one I have created Unit Tests to increase code coverage which I want to contribute. You can merge this pull request into a Git repository by running: $ git p

[GitHub] commons-imaging pull request #26: (doc) Update sample usage example links fr...

2017-07-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-imaging/pull/26 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] commons-imaging pull request #26: (doc) Update sample usage example links fr...

2017-07-10 Thread deroneriksson
GitHub user deroneriksson opened a pull request: https://github.com/apache/commons-imaging/pull/26 (doc) Update sample usage example links from svn to git This updates the broken example links at src/site/xdoc/sampleusage.xml so that the generated page at target/site/sampleusage.htm