Re: プラグインのライセンスは GPL にしましょう (pochy-user) - pochy

こんばんは。yhoです。

指摘ありがとうございます。
インラインで回答します。

On 2004 Feb 23 02:31:43 
clock <clock****@yahoo*****> wrote:

> --- yho <yhoyh****@plala*****> からのメッセージ：
> > こんばんは。yhoです。
> > 私が作成したPlguginもソースコードにはライセンス文を
> > いれていたのですが、公開していませんでしたので、
> > すべて公開しました。
> 
> ソースを眺めていて幾つか気になったところがあるので報告です。
> 
> FilterManager.cpp
> 152行目delete lpszHeaderはdelete [] lpszHeaderだと思います。
リークしていますね。
平中さんから頂いたWin98でコンボボックスのdropdownサイズが0になる件の
修正も行った、v0.1.2をリリースしました。

> ClearTrash.cpp
> 69行目 BOOL bとbが再定義されています。
> 変数のスコープと上書き時の動作について理解して書いているなら
> いいのですがわかりにくくなるので別名にしたほうがいいと思います。
> 
> 108行目FindCloseはハンドルが有効なときだけ呼んだほうがいいと
> 思います。
変数については、書いているときに気づきましたが、
やっぱりわかりにくいですかね。
FindClose共々修正しました。

> ClearTrashではPC_OnCloseの時に処理をしているようですが
> EndDialogPluginを使うと終了時でなくてもゴミ箱が空になることがあります。
> 注意書きを入れておいたほうがいいかもしれません。
はい、ここは気になっていました。ゴミ箱を開いているときに閉じる処理が
行われ、何かのプラグインでPC_OnCloseがキャンセルされると、メールは
消えているのに表示はそのままになってしまうので、PC_OnUnloadPluginで
ファイルを消すように修正しました。

上の修正を行ったClearTrash ｖ0.0.3をリリースしました。

以上、よろしくお願いします。
---
yho <yhoyh****@plala*****>

pochy

[pochy-user] Re: プラグインのライセンスは GPL にしましょう