svn commit: r1515039 - /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java
Author: bodewig Date: Sat Aug 17 19:46:10 2013 New Revision: 1515039 URL: http://svn.apache.org/r1515039 Log: count bytes read in ArjArchiveInputStream Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java URL: http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java?rev=1515039r1=1515038r2=1515039view=diff == --- commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java (original) +++ commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java Sat Aug 17 19:46:10 2013 @@ -95,10 +95,23 @@ public class ArjArchiveInputStream exten System.out.println(message); } -private static int read16(final DataInputStream in) throws IOException { +private int read8(final DataInputStream in) throws IOException { +int value = in.readUnsignedByte(); +count(1); +return value; +} + +private int read16(final DataInputStream in) throws IOException { final int value = in.readUnsignedShort(); +count(2); return Integer.reverseBytes(value) 16; } + +private int read32(final DataInputStream in) throws IOException { +final int value = in.readInt(); +count(4); +return Integer.reverseBytes(value); +} private String readString(final DataInputStream in) throws IOException { final ByteArrayOutputStream buffer = new ByteArrayOutputStream(); @@ -109,15 +122,21 @@ public class ArjArchiveInputStream exten return new String(buffer.toByteArray(), charset); } +private void readFully(final DataInputStream in, byte[] b) +throws IOException { +in.readFully(b); +count(b.length); +} + private byte[] readHeader() throws IOException { boolean found = false; byte[] basicHeaderBytes = null; do { int first = 0; -int second = in.readUnsignedByte(); +int second = read8(in); do { first = second; -second = in.readUnsignedByte(); +second = read8(in); } while (first != ARJ_MAGIC_1 second != ARJ_MAGIC_2); final int basicHeaderSize = read16(in); if (basicHeaderSize == 0) { @@ -126,8 +145,8 @@ public class ArjArchiveInputStream exten } if (basicHeaderSize = 2600) { basicHeaderBytes = new byte[basicHeaderSize]; -in.readFully(basicHeaderBytes); -final int basicHeaderCrc32 = Integer.reverseBytes(in.readInt()); +readFully(in, basicHeaderBytes); +final int basicHeaderCrc32 = read32(in); final CRC32 crc32 = new CRC32(); crc32.update(basicHeaderBytes); if (basicHeaderCrc32 == (int)crc32.getValue()) { @@ -160,12 +179,13 @@ public class ArjArchiveInputStream exten mainHeader.securityVersion = firstHeader.readUnsignedByte(); mainHeader.fileType = firstHeader.readUnsignedByte(); mainHeader.reserved = firstHeader.readUnsignedByte(); -mainHeader.dateTimeCreated = Integer.reverseBytes(firstHeader.readInt()); -mainHeader.dateTimeModified = Integer.reverseBytes(firstHeader.readInt()); -mainHeader.archiveSize = 0xL Integer.reverseBytes(firstHeader.readInt()); -mainHeader.securityEnvelopeFilePosition = Integer.reverseBytes(firstHeader.readInt()); +mainHeader.dateTimeCreated = read32(firstHeader); +mainHeader.dateTimeModified = read32(firstHeader); +mainHeader.archiveSize = 0xL read32(firstHeader); +mainHeader.securityEnvelopeFilePosition = read32(firstHeader); mainHeader.fileSpecPosition = read16(firstHeader); mainHeader.securityEnvelopeLength = read16(firstHeader); +pushedBackBytes(20); // count has already counted them via readFully mainHeader.encryptionVersion = firstHeader.readUnsignedByte(); mainHeader.lastChapter = firstHeader.readUnsignedByte(); @@ -183,8 +203,8 @@ public class ArjArchiveInputStream exten final int extendedHeaderSize = read16(in); if (extendedHeaderSize 0) { mainHeader.extendedHeaderBytes = new byte[extendedHeaderSize]; -in.readFully(mainHeader.extendedHeaderBytes); -final int extendedHeaderCrc32 = Integer.reverseBytes(in.readInt()); +readFully(in, mainHeader.extendedHeaderBytes); +final int extendedHeaderCrc32 = read32(in);
svn commit: r1515041 - /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveEntry.java
Author: bodewig Date: Sat Aug 17 20:15:27 2013 New Revision: 1515041 URL: http://svn.apache.org/r1515041 Log: the constants will be far more useful when public Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveEntry.java Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveEntry.java URL: http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveEntry.java?rev=1515041r1=1515040r2=1515041view=diff == --- commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveEntry.java (original) +++ commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveEntry.java Sat Aug 17 20:15:27 2013 @@ -92,18 +92,18 @@ public class ArjArchiveEntry implements * The known values for HostOs. */ public static class HostOs { -static final int DOS = 0; -static final int PRIMOS = 1; -static final int UNIX = 2; -static final int AMIGA = 3; -static final int MAC_OS = 4; -static final int OS_2 = 5; -static final int APPLE_GS = 6; -static final int ATARI_ST = 7; -static final int NEXT = 8; -static final int VAX_VMS = 9; -static final int WIN95 = 10; -static final int WIN32 = 11; +public static final int DOS = 0; +public static final int PRIMOS = 1; +public static final int UNIX = 2; +public static final int AMIGA = 3; +public static final int MAC_OS = 4; +public static final int OS_2 = 5; +public static final int APPLE_GS = 6; +public static final int ATARI_ST = 7; +public static final int NEXT = 8; +public static final int VAX_VMS = 9; +public static final int WIN95 = 10; +public static final int WIN32 = 11; } }
svn commit: r1515061 - /commons/proper/jci/trunk/pom.xml
Author: ebourg Date: Sun Aug 18 03:00:56 2013 New Revision: 1515061 URL: http://svn.apache.org/r1515061 Log: Upgraded to commons-parent 32 and removed the old release profile Modified: commons/proper/jci/trunk/pom.xml Modified: commons/proper/jci/trunk/pom.xml URL: http://svn.apache.org/viewvc/commons/proper/jci/trunk/pom.xml?rev=1515061r1=1515060r2=1515061view=diff == --- commons/proper/jci/trunk/pom.xml (original) +++ commons/proper/jci/trunk/pom.xml Sun Aug 18 03:00:56 2013 @@ -22,7 +22,7 @@ parent groupIdorg.apache.commons/groupId artifactIdcommons-parent/artifactId -version17/version +version32/version /parent packagingpom/packaging groupIdorg.apache.commons/groupId @@ -46,12 +46,7 @@ modulecompilers/rhino/module !-- modulecompilers/jsr199/module -- /modules -distributionManagement -site -idwebsite/id -urlscpexe://people.apache.org/www/commons.apache.org/jci//url -/site -/distributionManagement + developers developer idtcurdt/id @@ -202,91 +197,14 @@ /plugin /plugins /reporting -profiles -profile -idrelease/id -distributionManagement -repository -idapache.staging/id -nameApache Release Staging Repository/name - url${commons.deployment.protocol}://people.apache.org/www/people.apache.org/builds/commons/jci/${release.tag}//url -/repository -/distributionManagement -build -pluginManagement -plugins -plugin -groupIdorg.apache.maven.plugins/groupId -artifactIdmaven-source-plugin/artifactId -version2.0.3/version -/plugin -/plugins -/pluginManagement -plugins -plugin -artifactIdmaven-gpg-plugin/artifactId -version1.0-alpha-3/version -configuration -useAgenttrue/useAgent -/configuration -executions -execution -idsign-artifacts/id -phaseverify/phase -goals -goalsign/goal -/goals -/execution -/executions -/plugin -plugin -artifactIdmaven-source-plugin/artifactId -executions -execution -idcreate-source-jar/id -goals -goaljar/goal -/goals -/execution -/executions -/plugin -plugin -artifactIdmaven-javadoc-plugin/artifactId -executions -execution -idcreate-javadoc-jar/id -goals -goaljar/goal -/goals -configuration -source${maven.compile.source}/source -/configuration -/execution -/executions -/plugin -plugin -artifactIdmaven-release-plugin/artifactId -version2.0-beta-4/version -configuration -generateReleasePomstrue/generateReleasePoms -preparationGoalsclean install/preparationGoals -autoVersionSubmodulestrue/autoVersionSubmodules -tag${release.tag}/tag -/configuration -/plugin -/plugins -/build -/profile -/profiles + properties -gpg.useAgenttrue/gpg.useAgent maven.compile.source1.4/maven.compile.source maven.compile.target1.4/maven.compile.target commons.componentidjci/commons.componentid commons.release.version1.0/commons.release.version commons.jira.idJCI/commons.jira.id commons.jira.pid12310650/commons.jira.pid -commons.deployment.protocolscpexe/commons.deployment.protocol release.tag1.1-RC1/release.tag
svn commit: r1515065 - in /commons/proper/jci/trunk/compilers/janino: pom.xml src/main/java/org/apache/commons/jci/compilers/JaninoCompilationProblem.java src/main/java/org/apache/commons/jci/compiler
Author: ebourg Date: Sun Aug 18 03:53:22 2013 New Revision: 1515065 URL: http://svn.apache.org/r1515065 Log: Upgraded the dependency on Janino to 2.6.1 (JCI-69) Modified: commons/proper/jci/trunk/compilers/janino/pom.xml commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoCompilationProblem.java commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoJavaCompiler.java Modified: commons/proper/jci/trunk/compilers/janino/pom.xml URL: http://svn.apache.org/viewvc/commons/proper/jci/trunk/compilers/janino/pom.xml?rev=1515065r1=1515064r2=1515065view=diff == --- commons/proper/jci/trunk/compilers/janino/pom.xml (original) +++ commons/proper/jci/trunk/compilers/janino/pom.xml Sun Aug 18 03:53:22 2013 @@ -52,7 +52,7 @@ dependency groupIdorg.codehaus.janino/groupId artifactIdjanino/artifactId -version2.5.16/version +version2.6.1/version /dependency /dependencies properties Modified: commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoCompilationProblem.java URL: http://svn.apache.org/viewvc/commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoCompilationProblem.java?rev=1515065r1=1515064r2=1515065view=diff == --- commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoCompilationProblem.java (original) +++ commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoCompilationProblem.java Sun Aug 18 03:53:22 2013 @@ -18,8 +18,8 @@ package org.apache.commons.jci.compilers; import org.apache.commons.jci.problems.CompilationProblem; -import org.codehaus.janino.Location; -import org.codehaus.janino.util.LocatedException; +import org.codehaus.commons.compiler.LocatedException; +import org.codehaus.commons.compiler.Location; /** * Janino version of a CompilationProblem Modified: commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoJavaCompiler.java URL: http://svn.apache.org/viewvc/commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoJavaCompiler.java?rev=1515065r1=1515064r2=1515065view=diff == --- commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoJavaCompiler.java (original) +++ commons/proper/jci/trunk/compilers/janino/src/main/java/org/apache/commons/jci/compilers/JaninoJavaCompiler.java Sun Aug 18 03:53:22 2013 @@ -30,15 +30,14 @@ import org.apache.commons.jci.readers.Re import org.apache.commons.jci.stores.ResourceStore; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; +import org.codehaus.commons.compiler.CompileException; +import org.codehaus.commons.compiler.LocatedException; +import org.codehaus.commons.compiler.Location; import org.codehaus.janino.ClassLoaderIClassLoader; -import org.codehaus.janino.CompileException; import org.codehaus.janino.Compiler; -import org.codehaus.janino.DebuggingInformation; import org.codehaus.janino.FilterWarningHandler; -import org.codehaus.janino.Location; -import org.codehaus.janino.WarningHandler; import org.codehaus.janino.UnitCompiler.ErrorHandler; -import org.codehaus.janino.util.LocatedException; +import org.codehaus.janino.WarningHandler; import org.codehaus.janino.util.StringPattern; import org.codehaus.janino.util.resource.Resource; import org.codehaus.janino.util.resource.ResourceCreator; @@ -153,7 +152,9 @@ public final class JaninoJavaCompiler ex }, pSettings.getSourceEncoding(), false, - pSettings.isDebug()?DebuggingInformation.ALL:DebuggingInformation.NONE, + pSettings.isDebug(), +pSettings.isDebug(), +pSettings.isDebug(), new FilterWarningHandler(pattern, new WarningHandler() { public void handleWarning( final String pHandle, final String pMessage, final Location pLocation ) { final CompilationProblem problem = new JaninoCompilationProblem(pLocation.getFileName(), pLocation, pMessage, false);