allow volume argument to be 0-255 and convert 0-128 for Ruby/SDL API.

[junichi-ishikura]

Sound#set_volume( volume, time=0 ) を修正し、 0〜255 で指定されたボリュームを Ruby/SDL のAPIに合わせて 0〜128 に変換するようにしました。

[takaokouji]

あらためて見てみると volume * 128 / 255 が半端な値になることがあるため、四捨五入しましょうかね (volume * 128.0 / 255).round 。

また、下部でも同じ処理をしているので、メソッドにくくりだしましょうかね。

def dxruby_volume_to_sdl_volume(volume)
  (volume * 128.0 / 255).round
end

そして、 128 と 255 を定数にするとより良いでしょうね。

MAX_DXRUBY_VOLUME = 255
private_constant :MAX_DXRUBY_VOLUME
MAX_SDL_VOLUME = 128
private_constant :MAX_SDL_VOLUME

def dxruby_volume_to_sdl_volume(volume)
  (volume * MAX_SDL_VOLUME.to_f / MAX_DXRUBY_VOLUME).round
end

なお、 MAX_DXRUBY_VOLUME と MAX_SDL_VOLUMEをprivateスコープにしているのは、DXRubyではそれらを定義していないからです。dxruby_sdl では、極力、DXRubyが提供していないものは、提供しないようにしています。

[junichi-ishikura]

ボリュームの変換処理をくくりだす dxruby_volume_to_sdl_volume(volume)メソッドは、Soundクラスに定義してMusic#set_volumeやWave#set_volumeから使用するイメージですか:question:

[takaokouji]

dxruby_volume_to_sdl_volume(volume) は Common モジュールに定義して、 Music と Wave で Common を include することにしましょうかね。

[takaokouji]

LGTM