Skip to content

fpos_t and fread #1468

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

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

fpos_t and fread #1468

wants to merge 16 commits into from

Conversation

rotarymars
Copy link
Member

2つ作成してみました。確認していただけますか?

Copy link
Contributor

github-actions bot commented Jul 12, 2025

プレビュー (HTML) (更新時刻: 2025-07-15 08:35:15 JST)

  • Commit: 4124913
  • プレビューの生成には時間がかかります (3~5分)。進捗状況はこちらをご確認ください。

変更記事一覧

3件の記事が変更されました。

※ソース (.md) に直接変更のあった記事を列挙しています。グローバル修飾や変換規則の変更による変化は考慮していません。

@rotarymars
Copy link
Member Author

自分の手元で正しく表示されるように確認しました。
もし間違いがあったらすみません。

Copy link
Member

@akinomyoga akinomyoga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

更新ありがとうございます!

たびたびすみません。細かいですが:

rotarymars and others added 3 commits July 14, 2025 21:18
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
Co-authored-by: Koichi Murase <myoga.murase@gmail.com>
@rotarymars
Copy link
Member Author

ありがとうございます。確認しました。

Copy link
Member

@akinomyoga akinomyoga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

良いと思います!


```cpp
namespace std {
using fpos_t = implementation-defined;
Copy link
Member

@akinomyoga akinomyoga Jul 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントだけ。これはもう一つの PR #1471 で議論しているんですが、fpos_t は別に implementation-defined (処理系定義) ではないんですよね (C/C++ の規格を見た限りは。見逃しがなければ)。例えば、処理系依存ぐらいの意味で "implementation-defined" を使っているのだとしたら、処理系定義は処理系依存とは違います。

この PR で直す場合はそれでもいいし、あるいは、そうでなければ PR #1471 の方で拾って object-type に書き換えます。どちらでもOKです。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

分かりました。
fpos_tについてはこのPRで編集させていただきます。
4124913

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

今後のために知っておきたいのですが、「処理系【定義】」の場合にのみ implementation-definedを、それ以外のときはそれ以外の文言を検討した方がよいのですか?

Copy link
Member

@akinomyoga akinomyoga Jul 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いちばん中立的で何も言わないのが、規格に倣って "see below" ですね (「記事内で説明するので省略」って感じで)。そうでなければ "unspecified" が適切だろうと思います (追記: ただし C++ では implementation-defined の場合は unspecified とは書けない)。もっと具体的に書きたい場合は、規格の説明の中から語句を拾ってきて "xxx-type" とか "xxx-expression" とか、その値・型の種類を書くという感じになると思います (fpos_t の場合は "object type 云々" というのがC規格に書いてあります)。間違った説明でなければ、何れでも大丈夫です。

namespace std {
using fpos_t = object-type;
}
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
```
```
* object-type[italic]

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

Successfully merging this pull request may close these issues.

2 participants