Remove redundant 'break'. #13272

<!-- This issue tracker is only for technical issues related to Bitcoin Core. General bitcoin questions and/or support requests are best directed to the Bitcoin StackExchange at https://bitcoin.stackexchange.com. For reporting security issues, please read instructions at https://bitcoincore.org/en/contact/. If the node is "stuck" during sync or giving "block checksum mismatch" errors, please ensure your hardware is stable by running memtest and observe CPU temperature with a load-test tool such as linpack before creating an issue! -->

<!-- Describe the issue -->

version: 0.16.0 source: /src/httpserver.cpp line 196、199、202、205: dead code

switch (m) {
case HTTPRequest::GET:
    return "GET";
    break;
case HTTPRequest::POST:
    return "POST";
    break;
case HTTPRequest::HEAD:
    return "HEAD";
    break;
case HTTPRequest::PUT:
    return "PUT";
    break;

<!--- What behavior did you expect? -->

<!--- What was the actual behavior (provide screenshots if the issue is GUI-related)? -->

<!--- How reliably can you reproduce the issue, what are the steps to do so? -->

<!-- What version of Bitcoin Core are you using, where did you get it (website, self-compiled, etc)? -->

<!-- What type of machine are you observing the error on (OS/CPU and disk type)? -->

<!-- Any extra information that might be useful in the debugging process. -->

<!--- This is normally the contents of a `debug.log` or `config.log` file. Raw text or a link to a pastebin type site are preferred. -->

switch (m) { case HTTPRequest::GET: return "GET"; break; //dead code

It's a good C++ programming style and will be optimized by the compiler.

As pointed out by @ken2812221 this does not affect the resulting binary and is just a stylistic issue, not a bug.

yes, i agree this is a code quality issue. Do you think it need to be modified?

I think it would be a regression to make this change.