IonValue impls currently allow null symbol table reference. This adds complexity all over the place, and I don't think its necessary. At the very least, DOM nodes should have a system symbol table.
We should also allow standalone values to have local symtabs. This makes sense during iteration, as via IonSystem.iterate() as well as when building DOMs from a reader, as via IonSystem.newValue(IonReader). Otherwise these things are too detached from their original context.
When inserting such standalone values, the local symtab should be detached and replaced as appropriate for the new context. Must also check unknown symbols at that point.
Imported from ION-267,IONJAVA-116
IonValue impls currently allow null symbol table reference. This adds complexity all over the place, and I don't think its necessary. At the very least, DOM nodes should have a system symbol table.
We should also allow standalone values to have local symtabs. This makes sense during iteration, as via IonSystem.iterate() as well as when building DOMs from a reader, as via IonSystem.newValue(IonReader). Otherwise these things are too detached from their original context.
When inserting such standalone values, the local symtab should be detached and replaced as appropriate for the new context. Must also check unknown symbols at that point.
Imported from ION-267,IONJAVA-116