サクラエディタBBS

[8199] Re2:CSVモードで、Excel相当のデータの解釈をしてほしいです

▽ 2016/11/4 (金) 15:44:08 LR4

Mozilla/5.0 (Windows NT 6.1; rv:45.0) Gecko/20100101 Firefox/45.0

▼ もかさん
> 上記のサンプルバイナリ
> http://mocaskr.web.fc2.com/weekly/sakura_weekly.html
> sakura2-3-0-0r4119_fix_csv_escape_v0_2.zip

このサンプルバイナリですが、テキストの折り返し位置設定との絡みで
プログラムが無応答になることがあるようです。
自分の環境では以下の手順で無応答になります。

(1)sakura.iniが無い状態で起動（初回起動）
(2)タイプ「基本」の設定で「右端で折り返す」に設定
(3)エディタ画面を、80桁で折り返されるような幅にする
(4)サンプルバイナリに添付されているskrw_fix_csv_escape_v0_2.patchを開く

(4)の数秒後にマウスをビューにかざすとマウスポインタが
砂時計状態になっていて操作を受け付けません。

環境が違うとこの通りでは起きないかもしれませんが、
・マウスで画面の端を掴んで画面サイズをぐりぐり変化させる
・スクロールバーをマウスで掴んで上下スクロールさせる
・マウス左クリックでカーソル位置を移動する
などの操作をしているうちに再現するかも、です。

[8200] Re3:CSVモードで、Excel相当のデータの解釈をしてほしいです

▽ 2016/11/5 (土) 00:35:12 ばぼ

Mozilla/5.0 (Windows NT 6.3; WOW64; Trident/7.0; rv:11.0) like Gecko

▼ LR4さん
> このサンプルバイナリですが、テキストの折り返し位置設定との絡みで
> プログラムが無応答になることがあるようです。

v2系のいくつかの機能は、設定ファイルに設定値がない状態を想定していません。
なので、使い方が間違っていると思います。
どこかのOSベンダー風に言うと「仕様」です。
設定ファイルは、あらかじめ用意しておくのがベターです。

ちなみに「右端で折り返す」機能にはバグがあります。
私はあまり使わない機能なので再現条件を把握していませんが、
どうもレイアウト解析の構造に起因する問題のようです。

本家バイナリでも再現するようでしたら、
sourceforgeのバグレポートにあげると誰かが対応してくれると思います。
もか氏のところのアレはpreview版で本家とは別、らしいです。

[8201] Re4:CSVモードで、Excel相当のデータの解釈をしてほしいです

▽ 2016/11/5 (土) 03:21:15 LR4

Mozilla/5.0 (Windows NT 10.0; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0

▼ ばぼさん

少々、言葉足らずだったようで申し訳ありません。

> v2系のいくつかの機能は、設定ファイルに設定値がない状態を想定していません。
> なので、使い方が間違っていると思います。
> どこかのOSベンダー風に言うと「仕様」です。
> 設定ファイルは、あらかじめ用意しておくのがベターです。

(1)の初回起動後すぐに[ファイル]-[サクラエディタの全終了]して設定ファイルを作成しておき、その後再起動して(2)～(4)を実施しても症状は再現しています。

> 本家バイナリでも再現するようでしたら、
> sourceforgeのバグレポートにあげると誰かが対応してくれると思います。
> もか氏のところのアレはpreview版で本家とは別、らしいです。

現時点での本家バイナリでは再現しません。
本家の最近のソースに、もかさんのパッチを適用して手元でビルドしたバイナリで発生することは確認しています。

このパッチは本家の正規パッチ投稿場所（PatchUnicode）へも投稿されているようですので、正式取り込みが実施される前に修正していただければ、と思っています。

[8205] Re5:CSVモードで、Excel相当のデータの解釈をしてほしいです

▽ 2016/11/5 (土) 19:44:01 ばぼ

Mozilla/5.0 (Windows NT 6.3; WOW64; Trident/7.0; rv:11.0) like Gecko

▼ LR4さん
> 少々、言葉足らずだったようで申し訳ありません。

コメントしてといて放置もアレなので再現確認してみました。
クリッカブルURLのためのカーソル変更判定の中から
もか氏追加のコードが呼ばれていて、そこでハングしてますね。

sakura_core/doc/layout/CLayoutMgr.cpp

1019行目で終了条件に誤った値を設定しているのが原因。
const CLogicXInt nEnd = nDataLen + pcLayout->GetLogicOffset();

1021行目の↓の終了条件を満たさず、無限ループ（応答なし）。
for( ; i < nEnd; scan.Next(), i = scan.GetPos() )

とりあえず↓のようにしてビルドすれば「応答なし」は解消します。
const CLogicXInt nEnd = nDataLen;// + pcLayout->GetLogicOffset();

ソースにパッチを適用してビルドできるってことは開発系の人かと思います。
よければパッチ投稿してあげてください。
私はsakura開発チームの中の人じゃありませんので。

追記：
つか、中の人お願いします。

[8208] Re6:CSVモードで、Excel相当のデータの解釈をしてほしいです

▽ 2016/11/7 (月) 02:56:22 もか

Mozilla/5.0 (Windows NT 6.3; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0

お二方、ありがとうございます。
本家提出パッチおよびサンプルバイナリ更新しました。

[▼次のスレッド]

INCM/CMT
Cyclamen v3.81