遠藤です。
2011/10/8 yagit@mknada.sakura.ne.jp <yagit@mknada.sakura.ne.jp>:
吉田 様
パッチの評価及び貴重なアドバイスありがとうございます。感謝します。
イテレータの pre-increment については、ご指摘の通りです。
教えて頂いた事を踏まえ、近日中に修正したパッチを libreoffice@lists.freedesktop.org に投稿します。
最後に、事情でお返事が遅れてしまったことをお詫びします。
追伸:当 ML(discuss@ja.libreoffice.org) ではパッチを添付した投稿に失敗 してしまいました。
禁止されているのであれば、パッチを添付して投稿可能になる事を希望し ます。
もしもモデレータの方がご覧になっているのであれば、検討をお願いします。
確かに、当MLでは添付ファイルが使えませんでしたね。
以前バグ報告をしたときに、参考用に添付ファイルをつけた際に、
コケたことを思い出しました。
では。
(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に上げると限られた人間の視野にしか入っ
てこず、忘れられてしまう可能性がおおいにあります。なので出来るだけ
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
--
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
--
Yasunori Endo
5th grade student of Sendai National College of Technology, Hirose
--
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.