[Patch] SMTP.cpp: fix gcc6 build

[mt]

Gcc6 warns '-Werror=misleading-indentation', so add curly braces to fix it.

/home/haiku/haiku/haiku/src/add-ons/mail_daemon/outbound_protocols/smtp/SMTP.cpp: In member function 'status_t SMTPProtocol::Send(const char*, const char*, BPositionIO*)':
/home/haiku/haiku/haiku/src/add-ons/mail_daemon/outbound_protocols/smtp/SMTP.cpp:913:37: error: this 'else' clause does not guard... [-Werror=misleading-indentation]
                                     else
                                     ^~~~
/home/haiku/haiku/haiku/src/add-ons/mail_daemon/outbound_protocols/smtp/SMTP.cpp:916:6: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'else'
      ReportProgress (bufferLen,0);
      ^~~~~~~~~~~~~~
/home/haiku/haiku/haiku/src/add-ons/mail_daemon/outbound_protocols/smtp/SMTP.cpp:931:31: error: this 'else' clause does not guard... [-Werror=misleading-indentation]
                             } else
                               ^~~~
/home/haiku/haiku/haiku/src/add-ons/mail_daemon/outbound_protocols/smtp/SMTP.cpp:935:5: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'else'
     ReportProgress (bufferLen - 3,0);
     ^~~~~~~~~~~~~~
cc1plus: all warnings being treated as errors

[korli]

LGTM, but maybe needs another +1.

[axeld]

I think I'd prefer to move the send() call into either a local macro, or an extra method, like this:

ssize_t
SMTPProtocol::_SendData(const void* data, size_t length)
{
#ifdef USE_SSL
    if (use_ssl)
        return SSL_write(ssl, data, length);
#endif
    return send(fSocket, data, length);
}

[mt]

Replying to axeld:

I think I'd prefer to move the send() call into either a local macro, or an extra method, like this:

Hi axeld, I update patch.

[waddlesplash]

Applied in hrev50308. Thanks!