3 ms·
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 direc
by oefrha 5y 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.)
- Philip-J-Fry 5y agoYeah, you wouldn't be able to import this into a separate module and use it.
- crypt0lution 5y agoThat'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!
- deleted 5y ago[deleted]