Page MenuHome

Geometry Nodes: Add Brick Texture
ClosedPublic

Authored by Charlie Jolly (charlie) on Oct 7 2021, 8:32 PM.

Diff Detail

Repository
rB Blender

Event Timeline

Charlie Jolly (charlie) requested review of this revision.Oct 7 2021, 8:32 PM
Charlie Jolly (charlie) created this revision.
Jacques Lucke (JacquesLucke) requested changes to this revision.Oct 14 2021, 4:16 PM

Are you aware of any reason why the node has some inputs that are not sockets? Changing that is out of scope now, I'm just wondering.

source/blender/nodes/shader/nodes/node_shader_tex_brick.cc
139

Comment style.

No need for inline. Same below.

169

Comment style.

198

The blender:: part is not necessary.

This revision now requires changes to proceed.Oct 14 2021, 4:16 PM
Charlie Jolly (charlie) marked 3 inline comments as done.Oct 14 2021, 9:00 PM

Are you aware of any reason why the node has some inputs that are not sockets? Changing that is out of scope now, I'm just wondering.

@Thomas Dinges (dingto) I think you did the original port to Cycles. Can you answer @Jacques Lucke (JacquesLucke) question?

This revision is now accepted and ready to land.Oct 26 2021, 4:05 PM
This revision was automatically updated to reflect the committed changes.