Skip to content

Commit 33dfb36

Browse files
committed
Step 17: Remove unnecessary blank lines and comments
Removed: - Obsolete comment about removed functions - Redundant blank line before trak creation - Comment 'All boxes now have sizes, insert as-is' (obvious from code) Total reduction: 5 lines Size: 904 lines (was 909, -5 lines) Running total: -182 lines from original 1086 (-16.8%)
1 parent 7686aab commit 33dfb36

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

src/QuickTimeMuxer.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -605,7 +605,6 @@ std::vector<uint8_t> QuickTimeMuxer::createVideoTrackMoovBox(const std::vector<u
605605
.add32(2) // next_track_ID
606606
.build("mvhd");
607607

608-
// Build trak (Track)
609608
auto trak = createTrakBox(sps, pps, width, height, timescale, duration, frameCount);
610609

611610
// Assemble moov box: moov_header(8) + mvhd + trak
@@ -822,8 +821,6 @@ std::vector<uint8_t> QuickTimeMuxer::createStblBox(const std::vector<uint8_t>& s
822821
std::vector<uint8_t> stbl;
823822
write32(stbl, 0); // size placeholder
824823
stbl.insert(stbl.end(), {'s', 't', 'b', 'l'});
825-
826-
// All boxes now have sizes, insert as-is
827824
stbl.insert(stbl.end(), stsd.begin(), stsd.end());
828825
stbl.insert(stbl.end(), stts.begin(), stts.end());
829826
stbl.insert(stbl.end(), stss.begin(), stss.end());
@@ -858,8 +855,6 @@ void QuickTimeMuxer::write16(std::vector<uint8_t>& vec, uint16_t value) {
858855
}
859856
void QuickTimeMuxer::write8(std::vector<uint8_t>& vec, uint8_t value) { vec.push_back(value); }
860857

861-
// getNALTypeName and getCurrentTimestamp removed - were only used for debug logging
862-
863858
// Flush write buffer to disk (like old MP4Muxer)
864859
void QuickTimeMuxer::flushBufferToDisk(bool force) {
865860
if (m_writeBuffer.empty() || m_fd < 0) {

0 commit comments

Comments
 (0)