No problem.

Just an fyi, this line is still a problem:

faces[6][6] = faces[die1][die2];

You should read back through and think of why.

Yes, your seed should only be set once.

Also, while changing 6 to 7 "works" it isn't necessarily the right answer, as now you have an unused row and column in your array, which is a waste of memory.

Edit: In light of what jim said, yes, assigning to faces[7][7] is considered to result in undefined behaviour and thus the code is broken.

So, now you have a 7x7 array, when you needed a 6x6. You don't want to adjust the array size, you want to adjust the index in which you are inserting values. essentially, instead of the dice being numbered 1, 2, 3, 4, 5, 6, you could think of them as being numbered 0, 1, 2, 3, 4, 5, and just adjust the header when you print out your table. (0 would be 1, etc.)

Even more accurate I suppose would be doing as I have suggested, but think of 0 as 6. This would mean that you do not need the +1 when you use the rand function. You would get a number between 0 and 5 which would naturally map into the array you originally created.

Having something like a github or bitbucket account with code from your own projects on it can also be very good.

Hmm... the best hint I can give you at this point is remember that arrays are zero indexed. That means that the first item is at position 0 and the last one is at position 5. Therefore, position 6 is out of range.

Vim for life, unless I'm writing Java, in which case I'll use Eclipse or if I am using Qt in which case I'll use QtCreator