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

Jack o lantern #14

Closed
wants to merge 12 commits into from
Closed

Jack o lantern #14

wants to merge 12 commits into from

Conversation

Myu-Unix
Copy link

Hi celeron55 !

I've worked on the Jack O'Lantern again :

  • The block is now craftable by 2 Cactus & 1 Torch.
  • Face is always displayed on the front of the player.
  • Block is diffusing light like a torch.
  • images follow the global name convention already used.

Please let me know what you think and if changes must be made.

Cheers !

@celeron55 celeron55 closed this Apr 8, 2012
@Ekdohibs Ekdohibs mentioned this pull request Jun 30, 2014
AI-WAIFU added a commit to AI-WAIFU/minetest that referenced this pull request Apr 21, 2023
JosiahWI pushed a commit to JosiahWI/minetest that referenced this pull request Apr 18, 2024
* Support u8 / u32 indices

* Skip primitives without vertices

* Add support for non-indexed geometry & skipping primitives

* Fix possible memory leak on error

* Use SSkinnedMesh

* Check indices

* Properly mirror node hierarchy

* Update .gitignore

* Reorder includes

* Add some throws for logic errors

* Fix non-indexed geometry winding order, add unit test

* Address code review comments

* Add matrix transform unit test
JosiahWI pushed a commit to JosiahWI/minetest that referenced this pull request Apr 18, 2024
* Support u8 / u32 indices

* Skip primitives without vertices

* Add support for non-indexed geometry & skipping primitives

* Fix possible memory leak on error

* Use SSkinnedMesh

* Check indices

* Properly mirror node hierarchy

* Update .gitignore

* Reorder includes

* Add some throws for logic errors

* Fix non-indexed geometry winding order, add unit test

* Address code review comments

* Add matrix transform unit test
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.

None yet

2 participants