吉田 様
パッチの評価及び貴重なアドバイスありがとうございます。感謝します。
イテレータの pre-increment については、ご指摘の通りです。
教えて頂いた事を踏まえ、近日中に修正したパッチを
[email protected] に投稿します。
最後に、事情でお返事が遅れてしまったことをお詫びします。
追伸：当 ML（[email protected]） ではパッチを添付した投稿に失敗
してしまいました。
禁止されているのであれば、パッチを添付して投稿可能になる事を希望し
ます。
もしもモデレータの方がご覧になっているのであれば、検討をお願いします。
(11/10/06 14:47), Kohei Yoshida wrote:
八木さん、
パッチを読みました。まだ個人的にパッチを当ててテストしてないので詳しい
フィードバックは出来ないですが、気がついた点だけ指摘させて下さい。
いわゆる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
--
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
