Skip to content

Commit

Permalink
Revert "Change IonSystemBuilder's default charset to UTF-8 (#802)"
Browse files Browse the repository at this point in the history
This reverts commit 5158706.
  • Loading branch information
tgregg committed Apr 29, 2024
1 parent 38e9423 commit 965eb8a
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions src/main/java/com/amazon/ion/system/IonSystemBuilder.java
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ public static IonSystemBuilder standard()
IonCatalog myCatalog;
boolean myStreamCopyOptimized = false;

IonTextWriterBuilder textWriterBuilder = IonTextWriterBuilder.standard();
IonTextWriterBuilder textWriterBuilder = IonTextWriterBuilder.standard().withCharsetAscii();
IonBinaryWriterBuilder binaryWriterBuilder = IonBinaryWriterBuilder.standard();
IonReaderBuilder readerBuilder = IonReaderBuilder.standard();

Expand Down Expand Up @@ -269,7 +269,8 @@ public final IonSystemBuilder withStreamCopyOptimized(boolean optimized)

/**
* Gets the text writer builder whose options will be used when building an
* {@link IonSystem}. By default, {@link IonTextWriterBuilder#standard()} will be used.
* {@link IonSystem}. By default, {@link IonTextWriterBuilder#standard()}
* using {@code US-ASCII} encoding will be used.
*
* @see #setIonTextWriterBuilder(IonTextWriterBuilder)
* @see #withIonTextWriterBuilder(IonTextWriterBuilder)
Expand All @@ -283,7 +284,8 @@ public final IonTextWriterBuilder getIonTextWriterBuilder() {
* an {@link IonSystem}.
*
* @param builder the writer builder to use in built systems.
* If unset, each system will be built with {@link IonTextWriterBuilder#standard()}.
* If unset, each system will be built with {@link IonTextWriterBuilder#standard()}
* using {@code US-ASCII} encoding.
*
* @see #getIonTextWriterBuilder()
* @see #withIonTextWriterBuilder(IonTextWriterBuilder)
Expand All @@ -303,7 +305,8 @@ public final void setIonTextWriterBuilder(IonTextWriterBuilder builder) {
* always be used instead.
*
* @param builder the writer builder to use in built systems.
* If unset, each system will be built with {@link IonTextWriterBuilder#standard()}.
* If unset, each system will be built with {@link IonTextWriterBuilder#standard()}
* using {@code US-ASCII} encoding.
*
* @see #getIonTextWriterBuilder()
* @see #setIonTextWriterBuilder(IonTextWriterBuilder)
Expand Down

0 comments on commit 965eb8a

Please sign in to comment.