ReadNextFrame() method do not change stream position

[patachi]

hmm ... maybe there should be another approach to achieve this.
in this code reader.CurrentTime has fixed value .

public static byte[] CropMp3(Mp3CropInfo_DTO cropInfo, bool includeId3Tags = false)
        {
            using (var source = new MemoryStream(cropInfo.Mp3File))
            using (var reader = new Mp3FileReader(source))
            using (var writer = new MemoryStream())
            {
                //Extract ID3 tags
                if (reader.Id3v2Tag != null && includeId3Tags)
                {
                    writer.Write(reader.Id3v2Tag.RawData, 0, reader.Id3v2Tag.RawData.Length);
                }
                //Optional :/
                //reader.CurrentTime = cropInfo.StartTime;
                Mp3Frame frame;
                while (reader.CurrentTime.CompareTo(cropInfo.EndTime) <= 0 && (frame = reader.ReadNextFrame()) != null)
                {
                    if (reader.CurrentTime.CompareTo(cropInfo.StartTime) >= 0)
                    {
                        writer.Write(frame.RawData, 0, frame.RawData.Length);
                    }
                }
                writer.Flush();
                return writer.ToArray();
            }
        }

[markheath]

thanks for the reminder, have been meaning to sort that out for a while

[philkild]

@markheath could you publish a new NuGet with this fix included?

[markheath]

yes, have just pushed 1.8.1