Commit e5bb4e86 authored by Steve Holme's avatar Steve Holme
Browse files

email: Simplified nesting of if statements in *_doing() functions

parent 6d8d3069

lib/imap.c

+3 −5
Original line number Diff line number Diff line
@@ -2194,13 +2194,11 @@ static CURLcode imap_doing(struct connectdata *conn, bool *dophase_done) 


  if(result)

    DEBUGF(infof(conn->data, "DO phase failed\n"));

  else {

    if(*dophase_done) {

  else if(*dophase_done) {

    result = imap_dophase_done(conn, FALSE /* not connected */);



    DEBUGF(infof(conn->data, "DO phase is complete\n"));

  }

  }



  return result;

}

lib/pop3.c

+3 −5
Original line number Diff line number Diff line
@@ -1566,13 +1566,11 @@ static CURLcode pop3_doing(struct connectdata *conn, bool *dophase_done) 


  if(result)

    DEBUGF(infof(conn->data, "DO phase failed\n"));

  else {

    if(*dophase_done) {

  else if(*dophase_done) {

    result = pop3_dophase_done(conn, FALSE /* not connected */);



    DEBUGF(infof(conn->data, "DO phase is complete\n"));

  }

  }



  return result;

}

lib/smtp.c

+3 −5
Original line number Diff line number Diff line
@@ -1565,13 +1565,11 @@ static CURLcode smtp_doing(struct connectdata *conn, bool *dophase_done) 


  if(result)

    DEBUGF(infof(conn->data, "DO phase failed\n"));

  else {

    if(*dophase_done) {

  else if(*dophase_done) {

    result = smtp_dophase_done(conn, FALSE /* not connected */);



    DEBUGF(infof(conn->data, "DO phase is complete\n"));

  }

  }



  return result;

}