You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "drmingdrmer (via GitHub)" <gi...@apache.org> on 2024/11/18 04:07:24 UTC

[PR] refactor: MaybeSend does not need to be unsafe [opendal]

drmingdrmer opened a new pull request, #5338:
URL: https://github.com/apache/opendal/pull/5338

   # Which issue does this PR close?
   
   Remove `unsafe` from trait `unsafe trait MaybeSend`
   
   Closes #.
   
   # Rationale for this change
   
   A safe trait provides the same functionality as its unsafe counterpart but requires no safety guarantees from the implementor, making the unsafe keyword unnecessary.
   
   # What changes are included in this PR?
   
   Remove `unsafe` from the trait declaration.
   
   # Are there any user-facing changes?
   
   This change is backward compatible
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] refactor(core): MaybeSend does not need to be unsafe [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo merged PR #5338:
URL: https://github.com/apache/opendal/pull/5338


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org