Summary

Simplified the event deserialization code by removing intermediate variables and directly inlining deserialization calls. This refactoring reduces code verbosity while maintaining the same functionality. Additionally it removes the possible shadowing of vars by the macro generated code.

Type of change

Please check one:

• Bug fix (fixes incorrect behavior)
• New feature
• Performance improvement
• Documentation change with concrete technical impact
• Style, wording, formatting, or typo-only change

Why is this change needed?

The event deserialization code was unnecessarily verbose, with multiple intermediate variables and separate functions that made the code harder to read and maintain. This change streamlines the code by directly using the deserialization results where needed, making the generated code more concise.

What was the behavior or documentation before?

Previously, the code would generate separate statements for deserializing each field and then use those variables in the constructor. For example:

let x = core::serde::Serde::deserialize(ref data)?;
let data = core::serde::Serde::deserialize(ref keys)?;
Option::Some(A {x, data, })

What is the behavior or documentation after?

Now, the deserialization is inlined directly in the struct initialization, making the code more concise:

Option::Some(A {
    x: core::serde::Serde::deserialize(ref data)?, 
    data: core::serde::Serde::deserialize(ref keys)?, 
})

Additional context

This change introduces a new helper function deserialize_member_code that returns the appropriate deserialization code string based on the field kind, which is then used directly in the struct initialization. This approach reduces code duplication and makes the generated code more maintainable.