search

GPUOpen-Effects / TressFX

DirectX 12 and Vulkan libraries that provide convenient access to realistically rendered and simulated hair and fur
MIT License
789 stars 131 forks source link

typos #15

Closed RoyLab closed 8 years ago

RoyLab commented 8 years ago

TressFX\amd_tressfx\src\TressFXSimulation.cpp L256, there should be 'which' not 'wich'

mrgreywater commented 8 years ago

There are lots of typos in the comments, but they don't really matter. If they bother you that much, you should go over the project, fix them and create a PR.

To name a few others: "assgin" Ln219 SceneRender.cpp "corrisponding" Ln321 SceneRender.cpp "suure" Ln808 SceneRender.cpp "bufferss" Ln199 TressFXSimulation.cpp "pixelinked" Ln111 TressFXRenderer.cpp "exsist" Ln745 TressFXRenderer.cpp "unecssary" Ln887 TressFXRenderer.cpp ...

jstewart-amd commented 8 years ago

I cleaned this up. I'll push the commit in a minute. We do take PRs for typos, though, as @mrgreywater indicated. That class of pull request is a low-risk way to contribute, if you are interested in the future.

mrgreywater commented 8 years ago

LengthConstriantsWindAndCollision https://github.com/GPUOpen-Effects/TressFX/blob/master/amd_tressfx/src/Shaders/TressFXSimulation.hlsl#L728