From f2286f8ae5a562108d77a7bf2ecfdd7d78fee01f Mon Sep 17 00:00:00 2001 From: Devin Ragotzy Date: Mon, 12 Apr 2021 15:50:34 -0400 Subject: [PATCH] Fix confusing variable naming in ruma-api-macros --- ruma-api-macros/src/api/parse.rs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/ruma-api-macros/src/api/parse.rs b/ruma-api-macros/src/api/parse.rs index 2570a1d0..946d0118 100644 --- a/ruma-api-macros/src/api/parse.rs +++ b/ruma-api-macros/src/api/parse.rs @@ -27,14 +27,15 @@ impl Parse for Api { fn parse(input: ParseStream<'_>) -> syn::Result { let metadata: Metadata = input.parse()?; - let attributes = input.call(Attribute::parse_outer)?; + let req_attrs = input.call(Attribute::parse_outer)?; let (request, attributes) = if input.peek(kw::request) { - let request = parse_request(input, attributes)?; - let attributes = input.call(Attribute::parse_outer)?; + let request = parse_request(input, req_attrs)?; + let after_req_attrs = input.call(Attribute::parse_outer)?; - (Some(request), attributes) + (Some(request), after_req_attrs) } else { - (None, attributes) + // There was no `request` field so the attributes are for `response` + (None, req_attrs) }; let response = if input.peek(kw::response) {