helpGNU ccRTP - Support: sr #105156, Make the pool dispatch packets...

 
 

sr #105156: Make the pool dispatch packets according to timestamp

Submitted by:  Jörgen Terner <jorter>
Submitted on:  Thu 16 Feb 2006 08:00:58 AM UTC  
 
Category: NonePriority: 5 - Normal
Severity: 1 - WishStatus: Done
Privacy: PublicAssigned to: None
Open/Closed: OpenOperating System: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 16 May 2006 12:58:55 PM UTC, comment #1:

Yes, it does make a lot more sense to schedule packets by timeout.
I've just applied your patch and will commit it to CVS in a few
minutes, as soon as I integrate your next patch for thread
safety :)

Sorry for the delay and thanks!

Federico Montesino Pouzols <fedemp>
Project Member
Thu 16 Feb 2006 08:00:58 AM UTC, original submission:

Now, the RTPSessionPool schedules all waiting packets immediately.

Why not wait for the scheduling timeout as the SingleThreadRTPSession does?

Something like in the attached file perhaps. I've "merged" the code from SingleThreadRTPSession that handles the timestamp-checks into the RTPSessionPool:s run-method. It seams to work fine and like this it is really easy to modify the code to use either a pool of sessions or separate threads for all sessions.

Note that I'm currently using ccrtp-1.3.4.

Jörgen Terner <jorter>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #3918:  pool.cpp added by jorter (7kB - text/plain - Modified RTPSessionPool to dispatch packets according to timestamp)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 2 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Tue 16 May 2006 12:58:55 PM UTCfedempStatusNone=>Done
Thu 16 Feb 2006 08:00:58 AM UTCjorterAttached File-=>Added pool.cpp, #206

Back to the top


Powered by Savane 3.1-cleanup