@@ -450,7 +450,7 @@ impl<R: Seek> Seek for BufReader<R> {
450450pub struct BufWriter < W : Write > {
451451 // FIXME: Can this just be W, instead of Option<W>? I don't see any code
452452 // paths that lead to this being None, or that ever check if it IS none,
453- // even in drop implementations.
453+ // even in drop implementations. #72925.
454454 inner : Option < W > ,
455455 // FIXME: Replace this with a VecDeque. Because VecDeque is a Ring buffer,
456456 // this would enable BufWriter to operate without any interior copies.
@@ -715,7 +715,7 @@ impl<W: Write> Write for BufWriter<W> {
715715 if self . buf . len ( ) + buf. len ( ) > self . buf . capacity ( ) {
716716 self . flush_buf ( ) ?;
717717 }
718- // FIXME: Why no len > capacity? Why not buffer len == capacity?
718+ // FIXME: Why no len > capacity? Why not buffer len == capacity? #72919
719719 if buf. len ( ) >= self . buf . capacity ( ) {
720720 self . panicked = true ;
721721 let r = self . get_mut ( ) . write ( buf) ;
@@ -735,7 +735,7 @@ impl<W: Write> Write for BufWriter<W> {
735735 if self . buf . len ( ) + buf. len ( ) > self . buf . capacity ( ) {
736736 self . flush_buf ( ) ?;
737737 }
738- // FIXME: Why no len > capacity? Why not buffer len == capacity?
738+ // FIXME: Why no len > capacity? Why not buffer len == capacity? #72919
739739 if buf. len ( ) >= self . buf . capacity ( ) {
740740 self . panicked = true ;
741741 let r = self . get_mut ( ) . write_all ( buf) ;
@@ -752,7 +752,7 @@ impl<W: Write> Write for BufWriter<W> {
752752 if self . buf . len ( ) + total_len > self . buf . capacity ( ) {
753753 self . flush_buf ( ) ?;
754754 }
755- // FIXME: Why no len > capacity? Why not buffer len == capacity?
755+ // FIXME: Why no len > capacity? Why not buffer len == capacity? #72919
756756 if total_len >= self . buf . capacity ( ) {
757757 self . panicked = true ;
758758 let r = self . get_mut ( ) . write_vectored ( bufs) ;
0 commit comments