Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

バージョンダイアログの "Share Ver" に関して #447

Closed
m-tmatma opened this issue Sep 16, 2018 · 11 comments
Closed

バージョンダイアログの "Share Ver" に関して #447

m-tmatma opened this issue Sep 16, 2018 · 11 comments
Milestone

Comments

@m-tmatma
Copy link
Member

バージョンダイアログの Share Ver で N_SHAREDATA_VERSION の値を表示しているが、

Share Ver が何を意味しているのかわからないので、共有メモリのバージョンとするか、
N_SHAREDATA_VERSION という内部で使用している名前をそのまま表示するとかした方がいいと思う。

@berryzplus
Copy link
Contributor

Share Ver については「外部連携用のバージョン」という認識です。

サクラエディタは自分自身を複数起動するマルチプロセスアプリです。
自分自身と通信するにもプロセス間通信が必要になる都合で、内部定数を公開しています。

Share Ver が何を意味しているのかわからないので、共有メモリのバージョンとするか、
N_SHAREDATA_VERSION という内部で使用している名前をそのまま表示するとかした方がいいと思う。

自分なりに解釈してみました。

  • バージョンダイアログの表示項目名がわかりにくくないか?
  • 分かりやすい名前に変えたほうがいいんじゃないか?
  • 適切な名前を選ぶのが難しければ、内部名を表示したらいいんじゃないか?

感じ方はひとそれぞれなので「なんとも」です。

  • Share Ver でいいんじゃない?
  • 十分意味が通ると思うので変えなくてよくない?
  • N_SHAREDATA_VERSION を表示するは避けるべきです

N_SHAREDATA_VERSION を表示したら、「分かりにくくない?」→「名前変えましょう。」→「変えます。」→「内部に同じ文言があるのでついでに変えます」の流れになりそう。

変えたい、変えたくない、の中間をとって ShareData Ver にするのは許容です。

Share という名前は不適切なアプリを連想させるので表示するのをやめましょう、なら同意です。

@m-tmatma
Copy link
Member Author

まず、この表示が誰のための表示かというのがポイントだと思います。

@berryzplus
Copy link
Contributor

この表示が誰のための表示か

開発者向けという認識です。
デバッグに使える内部情報の一種だと考えています。

スクリプトマクロを組む人なら「最低限必要なバージョン」を確認するために使えるかもしれません。ユーザとしての立場で考えて、Share Ver を何かに使えるケースを思いつきません。(質問するときに書き添えるくらいじゃないでしょうか?)

@m-tmatma
Copy link
Member Author

スクリプトマクロを組む人なら「最低限必要なバージョン」を確認するために使えるかもしれません。

  • スクリプトマクロから取得できるのでしょうか?
  • N_SHAREDATA_VERSION は共有メモリのバージョンだと思いますが、スクリプトマクロから見たときの仕様変更と連動しているのでしょうか?

@berryzplus
Copy link
Contributor

スクリプトマクロから取得できるのでしょうか?

まえに掲示板で見かけた気がするので「ある」前提で書きました。
いちおう書く前にちらっとCSMacroManager.cppを眺めた感じでは、なさそうでした。
ないと断言できるほどちゃんと見てません。

突っ込みの意図が見えませんが、どうしたいですか?

@m-tmatma
Copy link
Member Author

突っ込みの意図が見えませんが、どうしたいですか?

開発者視点でも必要か? という確認です。

ユーザー視点で考えても、開発者視点で考えてもいらないのなら削除すればいいですし、
開発者視点で役にたつのなら、残してもいいですが、開発者以外のユーザーが混乱
しない表現にしたいです。

@berryzplus
Copy link
Contributor

開発者視点でも必要か? という確認です。

アプリバージョンで確認できるので「不要」の認識です。

@m-tmatma
Copy link
Member Author

#449 を投げました。

@berryzplus
Copy link
Contributor

(Share Ver は バージョンダイアログから消えました。)

(表示してても実は使い道がない、というのが削除理由です。)

@kobake
Copy link
Member

kobake commented Sep 16, 2018

アプリバージョンが違ってもshareバージョンは同一というケースもありますけどね。

終わった話のようなので、これ以上のことは言いませんが。

@ds14050
Copy link
Contributor

ds14050 commented Sep 16, 2018

以前は Share ver. が異なるサクラエディタを同時には起動できなかったはずで、その意味ではユーザー視点でも(よくよく詳しい人にとっては)意味がありました。今は異なる Share ver. のサクラエディタが共存できたと思ったので、用済みといえば用済みです。

これ以上ないほど恣意的な使用例を出すなら、Share ver. が同じサクラエディタを管理者としてと通常ユーザーとしてとで同時に起動することができないので、あえて Share ver. の異なる2種類を使い分けるという使い方が、机上では、考えられなくはないです。

以前は Share ver. が異なるサクラエディタを同時には起動できなかった

だいぶ遡りました>24557f0#diff-f44b69e01b43ef28b4fd772b04ea16e0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants