Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


八木さん、

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

いわゆる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

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.