Hacker News new | ask | show | jobs
by oefrha 1526 days ago
README shows various structs with zero exported fields, and example code working with those unexported fields which don't make sense and won't work unless directly placed in this package. There are other problems like non-idiomatic snake_case naming. I guess this is OP's golang learning project? (To be clear, making mistakes in the beginning is completely fine.)
2 comments

That's correct. I like to use go for image processing and wanted to apply some of the algorithms I made to video but did not find a simple way to do so. I set out to make a video I/O module.

I'm currently fixing all the non-idiomatic problems as well as adding some getters for each struct. Thanks for the feedback!

Yeah, you wouldn't be able to import this into a separate module and use it.