Skip to content

Naming Conventions #1

New issue

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

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

Already on GitHub? # to your account

Open
recp opened this issue Jul 30, 2020 · 2 comments
Open

Naming Conventions #1

recp opened this issue Jul 30, 2020 · 2 comments
Labels
help wanted Extra attention is needed

Comments

@recp
Copy link
Owner

recp commented Jul 30, 2020

// 1.
gpu_frontface(rce, GPUWindingCounterClockwise);
gpu_cullmode(rce, GPUCullModeBack);
gpu_renderstate(rce, renderState);
gpu_depthstencil(rce, depthStencilState);

// 2.      
GPUFrontFace(rce, GPUWindingCounterClockwise);
GPUCullMode(rce, GPUCullModeBack);
GPURenderState(rce, renderState);
GPUDepthStencil(rce, depthStencilState);

// 3.
gpuFrontFace(rce, GPUWindingCounterClockwise);
gpuCullMode(rce, GPUCullModeBack);
gpuRenderState(rce, renderState);
gpuDepthStencil(rce, depthStencilState);

// 4.
gpu_set_frontface(rce, GPUWindingCounterClockwise);
gpu_set_cullmode(rce, GPUCullModeBack);
gpu_set_renderstate(rce, renderState);
gpu_set_depthstencil(rce, depthStencilState);
     
// 5. 
GPUSetFrontFace(rce, GPUWindingCounterClockwise);
GPUSetCullMode(rce, GPUCullModeBack);
GPUSetRenderState(rce, renderState);
GPUSetDepthStencil(rce, depthStencilState);

// 6.
gpuSetFrontFace(rce, GPUWindingCounterClockwise);
gpuSetCullMode(rce, GPUCullModeBack);
gpuSetRenderState(rce, renderState);
gpuSetDepthStencil(rce, depthStencilState);

Which one?

@recp recp pinned this issue Jul 30, 2020
@recp recp added the help wanted Extra attention is needed label Jul 30, 2020
@explosion-mental
Copy link

I like 3

the first chunck , gpu, gotta be in lowercaps. if u r writting something with this lib, then the thinking would be:

gpu ... thinking ... functiontouse

the functiontouse should be title case, maybe there are similar functions, for example like ActionLower or ActionUpper.

Anyway, this is just my preference and feedback c:

@recp
Copy link
Owner Author

recp commented Sep 19, 2022

@explosion-mental many thanks for your feedback[s].

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants