Skip to content

Commit defdaf8

Browse files
committed
Inline read_one_message
It's only called in one place
1 parent e62b4df commit defdaf8

File tree

1 file changed

+11
-19
lines changed

1 file changed

+11
-19
lines changed

src/lib.rs

Lines changed: 11 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -545,27 +545,19 @@ impl InnerConnection {
545545
Ok(try_desync!(self, self.stream.flush()))
546546
}
547547

548-
fn read_one_message(&mut self) -> std_io::Result<Option<BackendMessage>> {
549-
debug_assert!(!self.desynchronized);
550-
match try_desync!(self, self.stream.read_message()) {
551-
NoticeResponse { fields } => {
552-
if let Ok(err) = DbError::new_raw(fields) {
553-
self.notice_handler.handle_notice(err);
554-
}
555-
Ok(None)
556-
}
557-
ParameterStatus { parameter, value } => {
558-
self.parameters.insert(parameter, value);
559-
Ok(None)
560-
}
561-
val => Ok(Some(val))
562-
}
563-
}
564-
565548
fn read_message_with_notification(&mut self) -> std_io::Result<BackendMessage> {
549+
debug_assert!(!self.desynchronized);
566550
loop {
567-
if let Some(msg) = try!(self.read_one_message()) {
568-
return Ok(msg);
551+
match try_desync!(self, self.stream.read_message()) {
552+
NoticeResponse { fields } => {
553+
if let Ok(err) = DbError::new_raw(fields) {
554+
self.notice_handler.handle_notice(err);
555+
}
556+
}
557+
ParameterStatus { parameter, value } => {
558+
self.parameters.insert(parameter, value);
559+
}
560+
val => return Ok(val)
569561
}
570562
}
571563
}

0 commit comments

Comments
 (0)