HackMD
  • Beta
    Beta  Get a sneak peek of HackMD’s new design
    Turn on the feature preview and give us feedback.
    Go → Got it
    • Beta  Get a sneak peek of HackMD’s new design
      Beta  Get a sneak peek of HackMD’s new design
      Turn on the feature preview and give us feedback.
      Go → Got it
      • Sharing Link copied
      • /edit
      • View mode
        • Edit mode
        • View mode
        • Book mode
        • Slide mode
        Edit mode View mode Book mode Slide mode
      • Note Permission
      • Read
        • Owners
        • Signed-in users
        • Everyone
        Owners Signed-in users Everyone
      • Write
        • Owners
        • Signed-in users
        • Everyone
        Owners Signed-in users Everyone
      • More (Comment, Invitee)
      • Publishing
        Please check the box to agree to the Community Guidelines.
        Everyone on the web can find and read all notes of this public team.
        After the note is published, everyone on the web can find and read this note.
        See all published notes on profile page.
      • Commenting Enable
        Disabled Forbidden Owners Signed-in users Everyone
      • Permission
        • Forbidden
        • Owners
        • Signed-in users
        • Everyone
      • Invitee
      • No invitee
      • Options
      • Versions and GitHub Sync
      • Transfer ownership
      • Delete this note
      • Template
      • Insert from template
      • Export
      • Dropbox
      • Google Drive Export to Google Drive
      • Gist
      • Import
      • Dropbox
      • Google Drive Import from Google Drive
      • Gist
      • Clipboard
      • Download
      • Markdown
      • HTML
      • Raw HTML
    Menu Sharing Help
    Menu
    Options
    Versions and GitHub Sync Transfer ownership Delete this note
    Export
    Dropbox Google Drive Export to Google Drive Gist
    Import
    Dropbox Google Drive Import from Google Drive Gist Clipboard
    Download
    Markdown HTML Raw HTML
    Back
    Sharing
    Sharing Link copied
    /edit
    View mode
    • Edit mode
    • View mode
    • Book mode
    • Slide mode
    Edit mode View mode Book mode Slide mode
    Note Permission
    Read
    Owners
    • Owners
    • Signed-in users
    • Everyone
    Owners Signed-in users Everyone
    Write
    Owners
    • Owners
    • Signed-in users
    • Everyone
    Owners Signed-in users Everyone
    More (Comment, Invitee)
    Publishing
    Please check the box to agree to the Community Guidelines.
    Everyone on the web can find and read all notes of this public team.
    After the note is published, everyone on the web can find and read this note.
    See all published notes on profile page.
    More (Comment, Invitee)
    Commenting Enable
    Disabled Forbidden Owners Signed-in users Everyone
    Permission
    Owners
    • Forbidden
    • Owners
    • Signed-in users
    • Everyone
    Invitee
    No invitee
       owned this note    owned this note      
    Published Linked with GitHub
    Like BookmarkBookmarked
    Subscribed
    • Any changes
      Be notified of any changes
    • Mention me
      Be notified of mention me
    • Unsubscribe
    Subscribe
    # Reading notes: RFC 3318, Field Projection Link to document: https://github.com/y86-dev/rfcs/blob/field-projection/text/3318-field-projection.md ###### tags: `reading-club` Leave questions, observations, discussion topics below. --- ## Topic name: prompt --- ## Composability (might be addressed in RFC text, not yet sure) pnkfelix: Can the projections described here be composed with themselves, such that e.g. `&MaybeUninit<Cell<Struct>>` can be projectred to `&MaybeUninit<Cell<Field>>`? Or does this only go "one level deep" ? vincenzopalazzo: this is unclear also to me, and it is unclear also how the solution proposed in the RFC works at implementation level https://github.com/y86-dev/rfcs/blob/field-projection/text/3318-field-projection.md#reference-level-explanation and "Supported pointers" Distinction between `P` pointer types and `Wrapper` types (both happen to include `Pin`) tmandry: Don't see a reason why not I guess ## How to express non-magic-projecting access? (might be addressed in RFC text, not yet sure) pnkfelix: If one of the types in question already a struct, do we need to continue to support the previous access? If so, how is that expressed ... via a new operator? Or by some thing analogous to UFCS, but now for field access rather than method calls? (E.g. how does one express what would today be denoted by `.inner` for a given `#[repr(transparent)]` type) pnkfelix: I could imagine support for the previous kind of access might still be expressable via `match`... oh lets ask about that too ... Update: RFC as written does not allow for the old semantics to be expressed via `let` and presumably via `match` either, see below pnkfelix: Maybe the intention is that you can get access to the original fields by first turning your `&Wrapper` into a `*const Wrapper` and then the `.`-projection operator acts on the `Wrapper` type, not on the projected type? Field access has higher priority than projection: https://github.com/y86-dev/rfcs/blob/field-projection/text/3318-field-projection.md#deref-and-derefmut pnkfelix: But then if I try to access some field `inner` (and that's in `Pin`) then we have privacy-dependent name resolution? Maybe we want/have this anyway. tmandry: Probably should be called out in the RFC. ## does this impact `match`/`let` semantics (answer: yes) Update: The intention *is* that this will affect bindings; `let` is demonstrated in the RFC text, and presumably `match` is affected as well. pnkfelix: I can understand how `foo.field` ends up projecting out the internals of `field` within `foo`. But does that also mean that this is impacted: ```rust= let foo: &Cell<Struct> = ...; match foo { Struct { field } => ... // does this *start working*, where `field` now has type `&Cell<Field>` ? } ``` https://github.com/y86-dev/rfcs/blob/field-projection/text/3318-field-projection.md#bindings pnkfelix: "Enum bindings cannot be supported with the wrappers MaybeUninit\<T> and Cell\<T>". What's the actual constraint here? eholk: They don't explain what to do about the UB here. pnkfelix: Seems like you get clear benefits from `.` and (things you can put on rhs of a let), don't know why they attempted to do matches in the first go-round. pnkfelix: Seems simplest to punt, but then again, maybe it's so valuable to match on an enum etc that we should work it out. ## Implementation details (vincenzopalazzo) Why project one filed if you can prepare the code to project with an infinite struct filed? The code could be ```rust #[project(Pin)] struct MyStruct { buf: [u8; 64], ptr: *const u8, _pin: PhantomPinned, } // gen by the macros trait ProjectPinForMyStruct { fn buf_as_pin(&self) -> Pin<[u8; 64]>; // not sure that compile :) fn ptr_as_pin(&self) -> Pin<....> } impl ProjectPinForMyStruct for MyStruct { fn buf_as_pin(&self) -> Pin<[u8; 64]>; // not sure that compile :) fn ptr_as_pin(&self) -> Pin<....> ... so on } ``` pnkfelix: (Something about disjoint fields, sorry bad notes) eholk: It also talks about [Arc projection](https://github.com/y86-dev/rfcs/blob/field-projection/text/3318-field-projection.md#out-of-scope-arc-projection) ## Relation to dyn* tmandry: The `NoMetadataPtr` trait and some operations proposed for projection look kind of like `dyn*` if you squint. Is there some kind of relationship here or is that just surface-level resemblance? eholk: I was wondering the same thing. pnkfelix: dyn* is not google-able :) eholk: I almost called in `dyn⭐` when I was making the parsing change tmandry: `dyn* Future` is pointer+vtable-sized object that implements `Future`; has a conversion trait like `IntoRawPointer` that looks just like `NoMetadataPtr`. tmandry: One possible difference is that `dyn*` _can_ represent `Box` (pnkfelix: also `Arc`) and ownership, whereas this is about borrowed data exclusively pnkfelix: More utility for this is projection of field where you erase the knowledge of original struct? tmandry: I guess I don't see a reason why they can't be the same trait. eholk: If you squint, dyn* might be able to represent a lot of these projection.

    Import from clipboard

    Advanced permission required

    Your current role can only read. Ask the system administrator to acquire write and comment permission.

    This team is disabled

    Sorry, this team is disabled. You can't edit this note.

    This note is locked

    Sorry, only owner can edit this note.

    Reach the limit

    Sorry, you've reached the max length this note can be.
    Please reduce the content or divide it to more notes, thank you!

    Import from Gist

    Import from Snippet

    or

    Export to Snippet

    Are you sure?

    Do you really want to delete this note?
    All users will lost their connection.

    Create a note from template

    Create a note from template

    Oops...
    This template is not available.


    Upgrade

    All
    • All
    • Team
    No template found.

    Create custom template


    Upgrade

    Delete template

    Do you really want to delete this template?

    This page need refresh

    You have an incompatible client version.
    Refresh to update.
    New version available!
    See releases notes here
    Refresh to enjoy new features.
    Your user state has changed.
    Refresh to load new user state.

    Sign in

    Forgot password

    or

    By clicking below, you agree to our terms of service.

    Sign in via Facebook Sign in via Twitter Sign in via GitHub Sign in via Dropbox

    New to HackMD? Sign up

    Help

    • English
    • 中文
    • Français
    • Deutsch
    • 日本語
    • Español
    • Català
    • Ελληνικά
    • Português
    • italiano
    • Türkçe
    • Русский
    • Nederlands
    • hrvatski jezik
    • język polski
    • Українська
    • हिन्दी
    • svenska
    • Esperanto
    • dansk

    Documents

    Tutorials

    Book Mode Tutorial

    Slide Mode Tutorial

    YAML Metadata

    Contacts

    Facebook

    Twitter

    Feedback

    Send us email

    Resources

    Releases

    Pricing

    Blog

    Policy

    Terms

    Privacy

    Cheatsheet

    Syntax Example Reference
    # Header Header 基本排版
    - Unordered List
    • Unordered List
    1. Ordered List
    1. Ordered List
    - [ ] Todo List
    • Todo List
    > Blockquote
    Blockquote
    **Bold font** Bold font
    *Italics font* Italics font
    ~~Strikethrough~~ Strikethrough
    19^th^ 19th
    H~2~O H2O
    ++Inserted text++ Inserted text
    ==Marked text== Marked text
    [link text](https:// "title") Link
    ![image alt](https:// "title") Image
    `Code` Code 在筆記中貼入程式碼
    ```javascript
    var i = 0;
    ```
    var i = 0;
    :smile: :smile: Emoji list
    {%youtube youtube_id %} Externals
    $L^aT_eX$ LaTeX
    :::info
    This is a alert area.
    :::

    This is a alert area.

    Versions

    Versions and GitHub Sync

    Sign in to link this note to GitHub Learn more
    This note is not linked with GitHub Learn more
     
    Add badge Pull Push GitHub Link Settings
    Upgrade now

    Version named by    

    More Less
    • Edit
    • Delete

    Note content is identical to the latest version.
    Compare with
      Choose a version
      No search result
      Version not found

    Feedback

    Submission failed, please try again

    Thanks for your support.

    On a scale of 0-10, how likely is it that you would recommend HackMD to your friends, family or business associates?

    Please give us some advice and help us improve HackMD.

     

    Thanks for your feedback

    Remove version name

    Do you want to remove this version name and description?

    Transfer ownership

    Transfer to
      Warning: is a public team. If you transfer note to this team, everyone on the web can find and read this note.

        Link with GitHub

        Please authorize HackMD on GitHub

        Please sign in to GitHub and install the HackMD app on your GitHub repo. Learn more

         Sign in to GitHub

        HackMD links with GitHub through a GitHub App. You can choose which repo to install our App.

        Push the note to GitHub Push to GitHub Pull a file from GitHub

          Authorize again
         

        Choose which file to push to

        Select repo
        Refresh Authorize more repos
        Select branch
        Select file
        Select branch
        Choose version(s) to push
        • Save a new version and push
        • Choose from existing versions
        Available push count

        Upgrade

        Pull from GitHub

         
        File from GitHub
        File from HackMD

        GitHub Link Settings

        File linked

        Linked by
        File path
        Last synced branch
        Available push count

        Upgrade

        Danger Zone

        Unlink
        You will no longer receive notification when GitHub file changes after unlink.

        Syncing

        Push failed

        Push successfully