@@ -450,7 +450,7 @@ impl<R: Seek> Seek for BufReader<R> {
450
450
pub struct BufWriter < W : Write > {
451
451
// FIXME: Can this just be W, instead of Option<W>? I don't see any code
452
452
// 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.
454
454
inner : Option < W > ,
455
455
// FIXME: Replace this with a VecDeque. Because VecDeque is a Ring buffer,
456
456
// this would enable BufWriter to operate without any interior copies.
@@ -715,7 +715,7 @@ impl<W: Write> Write for BufWriter<W> {
715
715
if self . buf . len ( ) + buf. len ( ) > self . buf . capacity ( ) {
716
716
self . flush_buf ( ) ?;
717
717
}
718
- // FIXME: Why no len > capacity? Why not buffer len == capacity?
718
+ // FIXME: Why no len > capacity? Why not buffer len == capacity? #72919
719
719
if buf. len ( ) >= self . buf . capacity ( ) {
720
720
self . panicked = true ;
721
721
let r = self . get_mut ( ) . write ( buf) ;
@@ -735,7 +735,7 @@ impl<W: Write> Write for BufWriter<W> {
735
735
if self . buf . len ( ) + buf. len ( ) > self . buf . capacity ( ) {
736
736
self . flush_buf ( ) ?;
737
737
}
738
- // FIXME: Why no len > capacity? Why not buffer len == capacity?
738
+ // FIXME: Why no len > capacity? Why not buffer len == capacity? #72919
739
739
if buf. len ( ) >= self . buf . capacity ( ) {
740
740
self . panicked = true ;
741
741
let r = self . get_mut ( ) . write_all ( buf) ;
@@ -752,7 +752,7 @@ impl<W: Write> Write for BufWriter<W> {
752
752
if self . buf . len ( ) + total_len > self . buf . capacity ( ) {
753
753
self . flush_buf ( ) ?;
754
754
}
755
- // FIXME: Why no len > capacity? Why not buffer len == capacity?
755
+ // FIXME: Why no len > capacity? Why not buffer len == capacity? #72919
756
756
if total_len >= self . buf . capacity ( ) {
757
757
self . panicked = true ;
758
758
let r = self . get_mut ( ) . write_vectored ( bufs) ;
0 commit comments