The StreamedMedia api is embarrassing. Let's not make the same mistake in Call git grep '<method' spec/*Call* shows the following questionable names: spec/Call_Content_Interface_Media.xml: <method name="Error" spec/Call_Stream_Interface_Media.xml: <method name="CandidatesPrepared" spec/Call_Stream_Interface_Media.xml: <method name="Error" git grep '<signal' spec/*Call* reveals only: spec/Call_Stream_Interface_Media.xml: <signal name="PleaseRestartICE" I plan to change PleaseRestartICE to ICERestartRequested (and have a boolean ICERestartPending for state recovery) as part of #35012 though, so no need to worry about that one. CandidatesPrepared could be called FinishInitialCandidateBatch, but that's a horrible name. As someone who has been screwed both by failing to call this method in sip and failing to flush sockets in unix, I'm tempted to call it FlushCandidates. Thoughts? The Error methods could both be called RemoveWithError.
Fail() instead of Error() ? InitialCandidatesDone() ? I don't think ICERestartPending is required.. Since if you've lost your state, you'll have to restart anyway
(In reply to comment #1) > Fail() instead of Error() ? > I don't think we're going to beat that for conciseness. > InitialCandidatesDone() ? > Still sounds like a signal name. Going for FlushInitialCandidates. http://cgit.collabora.com/git/user/alsuren/telepathy-spec.git/log/?h=call-trivia-38852 is ready for review.
Flush sounds like your throwing them away!.. And it's a signal really
(In reply to comment #3) > Flush sounds like your throwing them away!.. And it's a signal really The name was inspired by fflush. "CandidatesPrepared" only sounds like a signal (past tense) because it was copy-pasted from the (inside-out-sounding) StreamedMedia API. It is a method, which will cause rakia to do something important (but may be ignored by gabble). It should therefore use the imperative mood. I'm going to brainstorm a load of other possible names. Pick the one you like best. FinishInitialCandidates() SendInitialCandidates() # may be a lie if we don't have codecs yet. DoCandidatesDance() ProcessInitialCandidates() ProcessCandidates() # Probably the most honest name. Note that the purpose of this method is so that the CM can select its RawUDP candidate. If we just created a ProvideRawUDPCandidate() it would make the entire process more explicit.
The propose of this method is not just selecting the rawudp candidate. In SIP, you need to wait for it before you send out any the SDP offer with all the candidates. The least bad in your list is FinishInitialCandidates().. but really it is a signal, despite being a method call ...
Updated http://cgit.collabora.com/git/user/alsuren/telepathy-spec.git/log/?h=call-trivia-38852 to reflect this discussion.
++ SHIP IT!
merged to alsuren/call.
merged to master
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.