SpoutDev/Vanilla — comment on commit d10f698bc35f9e9648f8016ae7aab2b2b4900844
File:  line 0
By miketamis:

well if were trying imitate minecraft there is 3 different heights and the leaf are randomish. now this code is not as you said there are two types of sizes that are identical. my suggestion of how the code should be:

public void generateNormalTree(Chunk c, Point point) {
int x = (int) point.getX();
int y = (int) point.getY();
int z = (int) point.getZ();

+ short wood = VanillaBlocks.log.ge
int Hieght = random.nextInt(3) + 4;

generateNode(leaves,Hieght/2,hieght/4,x,y+Hieght,z);

for(int h = 0; h< Hieght; h++)
{
c.setBlockId(x, y+h, z, wood, world);
}

}
void generateNode(block, i,width, hieght,i,j,k);
{

}

it isnt perfect but you get an idea how i think it should be done, the node generation could be reused for all the tree.