遠藤です。 2011/10/8 [email protected] <[email protected]>: > 吉田 様 > > パッチの評価及び貴重なアドバイスありがとうございます。感謝します。 > > イテレータの pre-increment については、ご指摘の通りです。 > 教えて頂いた事を踏まえ、近日中に修正したパッチを [email protected] に投稿します。 > > 最後に、事情でお返事が遅れてしまったことをお詫びします。 > > 追伸：当 ML（[email protected]） ではパッチを添付した投稿に失敗 してしまいました。 > 禁止されているのであれば、パッチを添付して投稿可能になる事を希望し ます。 > もしもモデレータの方がご覧になっているのであれば、検討をお願いします。 確かに、当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に上げると限られた人間の視野にしか入っ >> てこず、忘れられてしまう可能性がおおいにあります。なので出来るだけ >> [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 > -- Yasunori Endo 5th grade student of Sendai National College of Technology, Hirose -- 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
