Re: [ja-discuss] 点線が使えない？

八木さん、

パッチを読みました。まだ個人的にパッチを当ててテストしてないので詳しい
フィードバックは出来ないですが、気がついた点だけ指摘させて下さい。

いわゆるi++やi--等のincrement/decrementですが、post-incrementよりも
pre-incrementを使うのがC++では推奨されています。これはパフォーマンスに関
わる事柄です。これが単純なintやlongのような数値の場合はコンパイラが最適
化してくれるのですが、iteratorの場合はそうは行かないのでiteratorを
incrementする際にはpre-increment (++i)でお願いします。単純な数値型の場合
でもpre-incrementを癖にしておくと一貫性があってコードも綺麗に見えます。

あと、パッチを上げる場合はbugzillaに上げると限られた人間の視野にしか入っ
てこず、忘れられてしまう可能性がおおいにあります。なので出来るだけ
libreoffice@lists.freedesktop.orgの方に、[PATCH]というタグをメールのサブ
ジェクトに入れて送って下さい。これはdrawing layerの変更なのでCedricより
もThorstenからチェックがくるかもしれません。僕も後でテストしてみたいと思
います。

ちなみに、パッチの場合はなるたけメールに添付してMLの方に送ってもらった方
がアーカイブにも残るのでそっちの方を推奨します。dropboxのような外部URLだ
といつか消えてしまうかもしれないので記録が残りません。

これはここのリストと、それとlibreoffice@lists.freedesktop.orgにも該当し
ます。

初めてパッチを上げる場合はライセンスについて必ず聞かれます。その際には、
もし特定のライセンスに固執しないのであれば、

My patch is being submitted under LGPLv3+/MPL.

と一言添えればライセンスに関するやりとりを省くことが出来、パッチの取り込
み作業がスムーズに進みます。

ではとりあえずそんなところで。頑張って下さい。

On Thu, 2011-10-06 at 08:16 +0900, yagit@mknada.sakura.ne.jp wrote:
> 皆様
> 八木 です。
>
> Calc／Writer で、点線／破線が描画されるようにするパッチを書きました。
> （geometry::ViewInformation2D クラスを利用します）
>
> パッチ内容及び報告内容（予定）について、もしもお気づきの点ありましたら教 
> えて下さい。
>
>
> パッチをここに置きました。
> http://dl.dropbox.com/u/20970054/fix-Dotted-and-Dashed-table-borders-are-not-drawn-correctly.patch


-- 
Kohei Yoshida, LibreOffice hacker, Calc


-- 
Unsubscribe instructions: E-mail to discuss+help@ja.libreoffice.org
Posting guidelines + more: http://wiki.documentfoundation.org/Netiquette
List archive: http://listarchives.libreoffice.org/ja/discuss/
All messages sent to this list will be publicly archived and cannot be deleted