八木さん、 パッチを読みました。まだ個人的にパッチを当ててテストしてないので詳しい フィードバックは出来ないですが、気がついた点だけ指摘させて下さい。
いわゆるi++やi--等のincrement/decrementですが、post-incrementよりも pre-incrementを使うのがC++では推奨されています。これはパフォーマンスに関 わる事柄です。これが単純なintやlongのような数値の場合はコンパイラが最適 化してくれるのですが、iteratorの場合はそうは行かないのでiteratorを incrementする際にはpre-increment (++i)でお願いします。単純な数値型の場合 でもpre-incrementを癖にしておくと一貫性があってコードも綺麗に見えます。 あと、パッチを上げる場合はbugzillaに上げると限られた人間の視野にしか入っ てこず、忘れられてしまう可能性がおおいにあります。なので出来るだけ [email protected]の方に、[PATCH]というタグをメールのサブ ジェクトに入れて送って下さい。これはdrawing layerの変更なのでCedricより もThorstenからチェックがくるかもしれません。僕も後でテストしてみたいと思 います。 ちなみに、パッチの場合はなるたけメールに添付してMLの方に送ってもらった方 がアーカイブにも残るのでそっちの方を推奨します。dropboxのような外部URLだ といつか消えてしまうかもしれないので記録が残りません。 これはここのリストと、それと[email protected]にも該当し ます。 初めてパッチを上げる場合はライセンスについて必ず聞かれます。その際には、 もし特定のライセンスに固執しないのであれば、 My patch is being submitted under LGPLv3+/MPL. と一言添えればライセンスに関するやりとりを省くことが出来、パッチの取り込 み作業がスムーズに進みます。 ではとりあえずそんなところで。頑張って下さい。 On Thu, 2011-10-06 at 08:16 +0900, [email protected] 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 [email protected] 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
