Possible feature request: Decode without clearing state about amf HOT 6 CLOSED

This is a somewhat interesting issue. Please let me consider the feasibility of this feature this weekend.

from amf.

Thank you very much!

from amf.

I found a description about reference table in AMF 3 specification as below.

Similar to AFM 0, AMF 3 object reference tables, object trait reference tables and string reference tables must be reset each time a new context header or message is processed.
-- 4.1 NetConnection and AMF 3

I think that it means that {amf0,amf3}::Decoder::decode methods should not clear the reference tables automatically after decoding a value (so the current implementation seems wrong...).

IMO, it seems better to split the current decode method into two methods as follows:

impl Decoder {
    pub fn decode(&mut self) -> DecodeResult<Value> {
        self.decode_value()
    }

   pub fn clear_reference_tables(&mut self) {
        self.traits.clear();
        self.strings.clear();
        self.complexes.clear();
   }
}

What do you think of it? > @crumblingstatue

from amf.

That looks good to me, it would be nice, thank you!

from amf.

Thank you for your swift reply. I created #5 to reflect the above idea.

from amf.

Published: https://crates.io/crates/amf/0.3.0

from amf.