BSDF sample refactor #4

- Fixed previous PR comments.
- Renamed BSDFSample::get_value() to value().
- BSDFSample now takes a ShadingPoint and modifies the shading basis automatically.
- Fix for an unrelated compiler error in appleseed.studio.

What about renaming this method to `get_shading_normal()` for the sake of consistency?