Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

gfxsort -> gfxutil #125

Closed
azul3d-bot opened this issue Mar 6, 2016 · 2 comments
Closed

gfxsort -> gfxutil #125

azul3d-bot opened this issue Mar 6, 2016 · 2 comments
Labels
enhancement gfx/gfxutil Long Term ready for release
Milestone
Version 2

Comments

@azul3d-bot
Copy link

Issue by slimsag
Saturday Dec 20, 2014 at 22:39 GMT
Originally opened as azul3d-legacy/gfx#72

gfxsort is an incredibly small and targeted package. This violates Go's practice of the fact that packages encompass an idea not a single task.

If we moved the sorting utilities to a gfx/gfxutil package instead, it would provide room for future expansion (e.g. gfxutil.LoadShader for loading shader files? who knows.)
@azul3d-bot
Copy link
Author

Comment by slimsag
Saturday Dec 20, 2014 at 23:55 GMT

Sorry for the accidental double-reference. I made the commits in the wrong branch and had to rebase.

@azul3d-bot azul3d-bot added this to the Version 2 milestone Mar 6, 2016
@azul3d-bot
Copy link
Author

Fixed/merged as part of #1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement gfx/gfxutil Long Term ready for release
Projects
None yet
Milestone
Version 2
Development

No branches or pull requests
1 participant
@azul3d-bot