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.

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

add: change shader feature #112

Closed
wants to merge 1 commit into from

Conversation

fum1h1ro
Copy link
Contributor

Unityで、LWFObjectに使用するシェーダ名を指定出来るようにしてみました。
よろしくお願いします。

@splhack
Copy link
Contributor

splhack commented Feb 12, 2015

LWFObjectのインスタンス変数で指定するよりも、Loadメソッドで指定したいです。

@fum1h1ro
Copy link
Contributor Author

なるほど。保存しておく必要は無いですしね。

@fum1h1ro fum1h1ro closed this Feb 13, 2015
@fum1h1ro
Copy link
Contributor Author

引数のどこに追加するかなんですが、末尾に追加すればいいかなと思ったところ、luaを使用している人が困るのではないかという問題が発生してしまいました。
これはオリジナルにはマージしない方が良いかもですね……。

@splhack
Copy link
Contributor

splhack commented Feb 13, 2015

いえ、luaStateの直前で大丈夫です! LWFObjectのサンプルでは、それにそなえて、あえて名前付き引数を使ってますので、大丈夫ということにしています。 Load("test.lwf", zRate=0.01f); など。

@fum1h1ro
Copy link
Contributor Author

なるほど了解です。名前付き引数便利ですね。
Booにはなかった……。
いやBoo自体消えそうですが('A`)

@fum1h1ro fum1h1ro mentioned this pull request Feb 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants