Sync code style in material.cpp

Update to use common code style.

No functional change.
Update SEE to return a Value

It seems more natural because the actual returned
value is from PieceValue[] array.

No functional change.
Merge remote-tracking branch 'upstream/master'