Sandertv / gophertunnel

General purpose library for Minecraft Bedrock Edition software written in Go
MIT License
419 stars 95 forks source link

Ability to implement your own reader and writer #183

Closed Flonja closed 1 year ago

Flonja commented 1 year ago

Note: ItemInstance and Item both have the NewReader/Writer and I'd like to hear if there would be a (better) way of handling that

Sandertv commented 1 year ago

I think this idea makes a lot of sense. I'm not sure how to solve that problem of ItemInstance and Item using NewReader and NewWriter though. Do you reckon that's a big issue?

Flonja commented 1 year ago

I think this idea makes a lot of sense. I'm not sure how to solve that problem of ItemInstance and Item using NewReader and NewWriter though. Do you reckon that's a big issue?

I don't think so, since a developer could reference their own IO if needed. But I'm still talking about the issue anyway, which is why this pull request is a draft.