[ja-discuss] Re: [ja-discuss] libvisio: 修正してみました

2014-11-07 スレッド表示 Isamu Mogi
茂木です。 ここはstd::maxがオススメです。 styleProps.insert(draw:marker-end-width, std::max(w, 0.05)); On 2014/11/07 17:00, Naruhiko Ogasawara wrote: 小笠原です。 小出しにごめんなさい。 1. ソース全部みたわけではないのですが変数 w の型は double というのは 間違いないんですよね。 2. ここで変数を宣言することはコーディング規約的に大丈夫なんでしたっけ。 というかDLPのコーディング規約はLibOと同じでいいのかな。 3.

Re: [ja-discuss] Re: [ja-discuss] libvisio: 修正してみました

2014-11-07 スレッド表示 TANAKA Hidemune
茂木さん、 ありがとうございます。 修正済みのDiffファイルを再送します。 On 2014年11月07日 17:24, Isamu Mogi wrote: ここは std::maxがオススメです。 styleProps.insert(draw:marker-end-width, std::max(w, 0.05)); -- ■□■□■□■□■□■□■□■□■□■□■□■  株式会社 田中コンピューターサービス 代表取締役 田中 秀宗   TANAKA Hidemune  本 社 東京都北区滝野川7丁目45番14号  電 話 

Re: [ja-discuss] Re: [ja-discuss] libvisio: 修正してみました

2014-11-07 スレッド表示 TANAKA Hidemune
添付ファイル使えないんでしたでしょうか。 メールに貼り付けて再送します。 diff --git a/src/lib/VSDContentCollector.cpp b/src/lib/VSDContentCollector.cpp index 9de0d30..a3cd0f1 100644 --- a/src/lib/VSDContentCollector.cpp +++ b/src/lib/VSDContentCollector.cpp @@ -2277,13 +2277,15 @@ void

[ja-discuss] Re: [ja-discuss] libvisio: 修正してみました

2014-11-07 スレッド表示 Naruhiko Ogasawara
小笠原です。 0.05 という値が妥当であるという理屈はありますか? あるなら、それをコメントなりで書くほうが適切だと思います。 私、以前は建設現場の現場監督やってたんですが、 AutoCADの線の太さに近づけるようにしました。 しっくりくるというか、見苦しくない太さに 調整したつもりです。 値の妥当性については特にコメントはないですが、最終的に パッチ送るときに、なんでそうしたんだと聞かれるとは確実に 思うので、その答えは準備したほうがいいと思います。 もちろん上の回答をそのまま伝えて皆が納得すればそれで よいですが。 では。 -- Naruhiko

Re: [ja-discuss] libvisio: 修正してみました

2014-11-07 スレッド表示 Isamu Mogi
茂木です。 AutoCADに近づけたとのことなので、いちおうVisioかVisio Viewerで表示した結果と だいたい一致するか確認すると良いと思いました。 もし既に確認済みだったらすみません。 On 2014/11/07 17:20, TANAKA Hidemune wrote: On 2014年11月07日 16:53, Naruhiko Ogasawara wrote: 0.05 という値が妥当であるという理屈はありますか? あるなら、それをコメントなりで書くほうが適切だと思います。 私、以前は建設現場の現場監督やってたんですが、

Re: [ja-discuss] libvisio: 修正してみました

2014-11-07 スレッド表示 TANAKA Hidemune
田中です。 比較用のファイルの場所も書いておきます。 http://tanaka-cs.co.jp/download/Visio.vsd http://tanaka-cs.co.jp/download/VisioPage1_1.JPG 以上、よろしくお願いします。 On 2014年11月08日 03:40, TANAKA Hidemune wrote: こ んにちは。田中です。 On 2014年11月07日 20:13, Isamu Mogi wrote: 茂木で す。 AutoCADに近づけたとのことなので、いちおうVisioかVisio Viewerで表示し