Skip to content

[diff.expr] Remove commentary about good practice in C#8356

Merged
jensmaurer merged 2 commits into
cplusplus:mainfrom
Keith-S-Thompson:void-star-conversion
Oct 24, 2025
Merged

[diff.expr] Remove commentary about good practice in C#8356
jensmaurer merged 2 commits into
cplusplus:mainfrom
Keith-S-Thompson:void-star-conversion

Conversation

@Keith-S-Thompson

Copy link
Copy Markdown
Contributor

C deliberately supports implicit conversion from void* to object*. Using an explicit cast (for assigning the result of malloc, for example) is not considered good practice in C, though it's required in C++.

I'm not aware of any C translators that warn about such assignments.

C deliberately supports implicit conversion from void* to object*.
Using an explicit cast (for assigning the result of malloc, for
example) is not considered good practice in C, though it's required
in C++.

I'm not aware of any C translators that warn about such assignments.
@frederick-vs-ja

Copy link
Copy Markdown
Contributor

This sentence can be found in the Jan 1994 draft. I guess SG22 should see this.

Comment thread source/compatibility.tex Outdated
Comment thread source/compatibility.tex Outdated
@jensmaurer

Copy link
Copy Markdown
Member

Please fix the CI failure:

2025-10-23T19:33:00.7284274Z ##[error]compatibility.tex:2999:use \Cpp{} instead: This is fairly widely used in C, but C++ requires a

@jensmaurer jensmaurer added the changes requested Changes to the wording or approach have been requested and not yet applied. label Oct 23, 2025
Remove unnecessary wording.

Co-authored-by: Jan Schultke <me@eisenwave.net>
@jensmaurer jensmaurer removed the changes requested Changes to the wording or approach have been requested and not yet applied. label Oct 24, 2025

@jensmaurer jensmaurer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

CWG 2025-10-24: Approved as an editorial change.

@jensmaurer jensmaurer merged commit cab90b3 into cplusplus:main Oct 24, 2025
2 checks passed
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.

4 participants