progenitor sometimes produces code which triggers clippy::unnecessary_to_owned

[Fishrock123]

    ///Swagger Endpoint
    ///
    ///Swagger docs (UI)
    ///
    ///Sends a `GET` request to `/docs/{proxy}`
    pub async fn get_swagger_docs_proxy<'a>(
        &'a self,
        proxy: &'a str,
    ) -> Result<ResponseValue<()>, Error<()>> {
        let url = format!("{}/docs/{}", self.baseurl, encode_path(&proxy.to_string()),);

Clippy output:

warning: unnecessary use of `to_string`
    --> src/codegen.rs:1636:67
     |
1636 |         let url = format!("{}/docs/{}", self.baseurl, encode_path(&proxy.to_string()),);
     |                                                                   ^^^^^^^^^^^^^^^^^^ help: use: `proxy`
     |
     = note: `#[warn(clippy::unnecessary_to_owned)]` on by default
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_to_owned

Swagger def:

    "/docs/{proxy}": {
      "get": {
        "tags": [
          "swagger"
        ],
        "summary": "Swagger Endpoint",
        "operationId": "getSwaggerDocsProxy",
        "description": "Swagger docs (UI)",
        "parameters": [
          {
            "name": "proxy",
            "in": "path",
            "required": true,
            "schema": {
              "type": "string"
            }
          }
        ],
        "responses": {
          "200": {
            "description": "swagger UI"
          }
        }
      }
    },

To be clar: this seems a bit worse than the other warnings. This is an extra unnecessary allocation.

[ahl]

Thanks for the bug and the simple reproduction case!

[ahl]

I think #1017 may inadvertently be addressing this.