Skip to content

Specify what happens when casting fat pointers more specifically#1206

Closed
CAD97 wants to merge 5 commits intorust-lang:masterfrom
CAD97:patch-1
Closed

Specify what happens when casting fat pointers more specifically#1206
CAD97 wants to merge 5 commits intorust-lang:masterfrom
CAD97:patch-1

Conversation

@CAD97
Copy link
Copy Markdown

@CAD97 CAD97 commented May 5, 2022

@ehuss ehuss added I-lang-nominated Nominated for discussion during a lang team meeting. T-lang Relevant to the language team. labels May 20, 2022
@scottmcm
Copy link
Copy Markdown
Member

@CAD97 Do you perhaps have a topic link for the discussion? The link seems to just be to the UCG stream in general.

Comment thread src/expressions/operator-expr.md
@joshtriplett joshtriplett removed the I-lang-nominated Nominated for discussion during a lang team meeting. label Jun 21, 2022
@joshtriplett
Copy link
Copy Markdown
Member

r=me with an example added.

@CAD97
Copy link
Copy Markdown
Author

CAD97 commented Jun 21, 2022

...oops, will grab when I find it again :)

@CAD97
Copy link
Copy Markdown
Author

CAD97 commented Jun 21, 2022

Comment thread src/expressions/operator-expr.md
@CAD97
Copy link
Copy Markdown
Author

CAD97 commented Aug 1, 2022

@joshtriplett I finally got around to adding that example 🙃 Is this still good to r+? (Well, after I add the proper uses.)

Comment thread src/expressions/operator-expr.md
Comment thread src/expressions/operator-expr.md Outdated
@CAD97
Copy link
Copy Markdown
Author

CAD97 commented Aug 2, 2022

That mess of commits shall be a lesson not to try to fix things through GitHub's app; rebased to fix that

@CAD97
Copy link
Copy Markdown
Author

CAD97 commented Aug 2, 2022

error in ../src/expressions/operator-expr.md: #![feature] attributes are not allowed

Well never mind then

Comment thread src/expressions/operator-expr.md Outdated
@CAD97
Copy link
Copy Markdown
Author

CAD97 commented Aug 2, 2022

This is just not my day for writing coherent code is it. The example should finally be accurate now...

Comment thread src/expressions/operator-expr.md
@traviscross
Copy link
Copy Markdown
Contributor

We reviewed this today in the lang-docs call, and this seems to have been superseded by:

...which was merged. So we'll go ahead and close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-lang Relevant to the language team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants