Matthew Pope

Results 70 comments of Matthew Pope

After discussing this with @zslayton, we realized that this also adds more complex control flow than we would like in TDL. Yes, you can test whether an argument is void,...

In https://github.com/amazon-ion/ion-tests/pull/133, we decided that a repeat of `0` is allowed, and confirmed that repeating an empty stream is also allowed.

Discussed this offline with @zslayton. Yes, anything that has the reserved system annotation (now `$ion`) that is not a valid system value should raise an error.

@barries, you might want to see https://github.com/amazon-ion/ion-docs/issues/397. We have been running into issues implementing the current Ion 1.1 specification, and we are likely to simplify things. How extensively have you...

@zslayton, would it be fair to summarize that as "increasing the MSRV requires a new minor version"? Another one that looks reasonable to me is [this proposal for libc](https://github.com/rust-lang/libs-team/issues/72).

Thank you for sharing your interest in an official Ion content type. For the time being, there is an existing _provisional_ media type `application/ion` that might be suitable for you...

@desaikd This PR was replaced by #941, right? Can we close this?

> Note the significant repetition of the pool pattern in PooledExpressionFactory. This is actually not easy to avoid without sacrificing performance (e.g. using lambdas, which require allocations and cut into...

>> Is there a concern about maintaining feature parity with ion libraries for other languages? > > I'd say no. But I will check with the other maintainers. _API_ feature...