You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Macro namespace conflicts will be resolved with macros 2.0, part of #19. As a result, namespacing macros in this way will be unnecessary. Furthermore, the aesthetics of what you've proposed are subpar. For these reasons, I am declining this request and closing the issue.
That being said, I have considered renaming error to catcher or catch to better declare its intention. This would inadvertently solve the problem.
@SergioBenitez I agree with switching to using #[catch]. Rust is adding catch as a contextual keyword so it seems to be the de-facto word for this kind of thing.
I was under the impression that macros (both attribute and functional) would move to using normal use instead of "magic" scope. This is still a conflict (though not as serious) as I'd imagine you'd start having use rocket::prelude::* recommended same as slog would recommend use slog::prelude::*.
Both
log
andslog
useerror!
for error logging. This conflicts with#[error]
as both attribute macros and functional macros share the same scope.I propose that we make it policy to namespace attribute macros outside of the standard HTTP methods and
route
.The text was updated successfully, but these errors were encountered: