comments

diff --git a/src/libFLAC/seekable_stream_encoder.c b/src/libFLAC/seekable_stream_encoder.c
index 320ad8d..bf04424 100644 (file)
--- a/src/libFLAC/seekable_stream_encoder.c
+++ b/src/libFLAC/seekable_stream_encoder.c
@@ -714,7 +714,7 @@ void metadata_callback_(const FLAC__StreamEncoder *encoder, const FLAC__StreamMe
 
        (void)encoder; /* silence compiler warning about unused parameter */
 
-       /*@@@ reopen callback here? */
+       /*@@@ reopen callback here?  The docs currently require user to open files in update mode from the start */
 
        /* All this is based on intimate knowledge of the stream header
         * layout, but a change to the header format that would break this

diff --git a/src/libFLAC/stream_decoder.c b/src/libFLAC/stream_decoder.c
index a5d575c..9a1672e 100644 (file)
--- a/src/libFLAC/stream_decoder.c
+++ b/src/libFLAC/stream_decoder.c
@@ -695,7 +695,7 @@ FLAC__bool allocate_output_(FLAC__StreamDecoder *decoder, unsigned size, unsigne
        if(size <= decoder->private_->output_capacity && channels <= decoder->private_->output_channels)
                return true;
 
-       /* @@@@ should change to use realloc() */
+       /* simply using realloc() is not practical because the number of channels may change mid-stream */
 
        for(i = 0; i < FLAC__MAX_CHANNELS; i++) {
                if(0 != decoder->private_->output[i]) {