日々の破片

_ VB -> COM -> VC++

なひさんのコメントを読んで思い出したまずいコード。

HRESULT Foo(VARIANT_BOOL bar)
{
    if (bar == TRUE)
    {
        // VBからは絶対実行されない。でもVC++からは呼べる
    }
    return S_OK;
}

まずいのは、呼び出す側のVC++プログラマも

IFoo* ifoo;
CoCreateInstance(CLSID_Foo, .... (void**)&ifoo);
ifoo->Foo(TRUE);
ifoo->Release();

なんて書いていたから。(確か、VBで作成したActiveXコントロールで、If bar = True Thenとか書いてあるとひっかっかたような気もするが不確実)
正しくは、

HRESULT Foo(VARIANT_BOOL bar)
{
    if (bar) // 勘違いしているVC++から呼ばれてもOK
    {
        // 正しく実行される
    }
    return S_OK;
}

で、呼び出しは

IFoo* ifoo;
CoCreateInstance(CLSID_Foo, .... (void**)&ifoo);
ifoo->Foo(VARIANT_TRUE);
ifoo->Release();

っていうか、Win32APIでも!FALSEが不定値というのは多いから、TRUEというdefineを無しにしておいて、

if (!Win32APICall(...)) // 失敗の判定
{
 ...
}
if (Win32APICall(...)) // 成功の判定
{
 ...
}

というのをベストプラクティスとしておきゃ良いのだが、なぜか、真かどうかを同値検査で判定するコードを書く人っているのだな、これが。

_ Subversion

これだけそこら中で見かけるのだから、よほどモノが良いか、よほど宣伝が良いかのどちらかだろう。で、当然、後者のはずはありえない。だから、機会を見つけて移行しようかな。

Subversion によるバージョン管理。

移行ツール次第のような。