165: loadImageにおける無駄なSDL_ConvertSurfaceの除去の提案

ONScripterLabel_image.cpp内の
ONScripterLabel::loadImage()内の
以下の部分についてですが

SDL_Surface *ret = SDL_ConvertSurface(tmp, image_surface->format, 
SDL_SWSURFACE);
SDL_FreeSurface(tmp);

もし、tmpのフォーマットがimage_surfaceのフォーマットと等しい場合
無駄な変換（単なるコピーだとは思いますが）をしてしまうことになります。
それにより、一時的にメモリを消費してしまうのと
読み込み速度がコピー速度だけ遅くなってしまうと思います。

例えば、createRectangleSurface(filename);で作成したサーフェスは
image_surfaceのフォーマットで作成されています。

ですので、上記の部分でフォーマットが一致した場合は
以下のようにSDL_ConvertSurfaceを呼ばないようにしてはどうでしょうか。

SDL_Surface *ret;
if((tmp->w * tmp->format->BytesPerPixel == tmp->pitch) &&
   (tmp->format->BitsPerPixel == image_surface->format->BitsPerPixel) && 
   (tmp->format->Rmask == image_surface->format->Rmask) &&
   (tmp->format->Gmask == image_surface->format->Gmask) &&
   (tmp->format->Bmask == image_surface->format->Bmask) &&
   (tmp->format->Amask == image_surface->format->Amask)){
	ret = tmp;
} else {
	ret = SDL_ConvertSurface(tmp, image_surface->format, SDL_SWSURFACE);
	SDL_FreeSurface(tmp);
}

秋月かたね

Amaskが0の場合を考えると一致しなくても
処理には問題なさそうなので訂正します。

SDL_Surface *ret;
if((tmp->w * tmp->format->BytesPerPixel == tmp->pitch) &&
   (tmp->format->BitsPerPixel == image_surface->format->BitsPerPixel) && 
   (tmp->format->Rmask == image_surface->format->Rmask) &&
   (tmp->format->Gmask == image_surface->format->Gmask) &&
   (tmp->format->Bmask == image_surface->format->Bmask)){
	ret = tmp;
} else {
	ret = SDL_ConvertSurface(tmp, image_surface->format, SDL_SWSURFACE);
	SDL_FreeSurface(tmp);
}

ご提案ありがとうございます。

遅くなりましたが、いただいた修正をそのまま取り入れさせていただきました。

20100808 以降で修正済みです。
ご確認ください。

これでSDL_image側を改造してONSと同じフォーマットで出力するようにして
画像の読み込み速度の高速化とメモリ節約をするバイナリを作れます。
非公式バイナリに適用する予定です。

修正ありがとうございました。

ご確認ありがとうございます。

標準の SDL_image を使用して jpg や bmp を読み込むと ONScripter で使用するフォーマ
ットとは異なるフォーマットになるため、内心実用上はあまり効果が無いかもしれないと
思っていたのですが、SDL_image 側を改造するのであれば効果が大きそうですね。

より使い勝手のよい ONScripter を公開していただけると、私もたいへんうれしいです。


それでは、本報告は完了とさせていただきます。