rtmpproto: Add a comment explaining the logic in handle_notify
This explains why the cleanup in5626f994fwas wrong and whyae0f316awas needed. Signed-off-by: Martin Storsjö <martin@martin.st>
This commit is contained in:
		@@ -2095,9 +2095,11 @@ static int handle_notify(URLContext *s, RTMPPacket *pkt) {
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
    // generate packet header and put data into buffer for FLV demuxer
 | 
					    // generate packet header and put data into buffer for FLV demuxer
 | 
				
			||||||
    if (rt->flv_off < rt->flv_size) {
 | 
					    if (rt->flv_off < rt->flv_size) {
 | 
				
			||||||
 | 
					        // There is old unread data in the buffer, thus append at the end
 | 
				
			||||||
        old_flv_size  = rt->flv_size;
 | 
					        old_flv_size  = rt->flv_size;
 | 
				
			||||||
        rt->flv_size += datatowritelength + 15;
 | 
					        rt->flv_size += datatowritelength + 15;
 | 
				
			||||||
    } else {
 | 
					    } else {
 | 
				
			||||||
 | 
					        // All data has been read, write the new data at the start of the buffer
 | 
				
			||||||
        old_flv_size = 0;
 | 
					        old_flv_size = 0;
 | 
				
			||||||
        rt->flv_size = datatowritelength + 15;
 | 
					        rt->flv_size = datatowritelength + 15;
 | 
				
			||||||
        rt->flv_off  = 0;
 | 
					        rt->flv_off  = 0;
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user