amazon-ion/ion-java

Revisit visibility changes and package locations of classes in #830

popematt opened this issue · 0 comments

Let's open an issue with a list of the classes we had to make public in this PR so we don't forget to go back and revisit them.

Originally posted by @tgregg in #830 (review)

The classes are:

  • com.amazon.ion.impl.LocalSymbolTable
  • com.amazon.ion.impl.bin.Block
  • com.amazon.ion.impl.bin.BlockAllocator
  • com.amazon.ion.impl.bin.BlockAllocatorProvider
  • com.amazon.ion.impl.bin.BlockAllocatorProviders

As part of revisiting the visibility of these classes, we also need to consider to package location of:

  • com.amazon.ion.impl.bin.IonManagedWriter_1_1
  • com.amazon.ion.impl.bin.ManagedWriterOptions_1_1
  • com.amazon.ion.impl.bin.DelimitedContainerStrategy
  • com.amazon.ion.impl.bin.SymbolInliningStrategy
  • com.amazon.ion.impl.BlockBufferingOutputStreamFastAppendable