|
Hi Vadim! I'm sending 2 small patches attached for approval, with respective change info as follows: "phapi-old.patch": changed the operator "==" to "=" on what seems to be a attribution statement; "phmedia.patch": changed the initialization of "ret2" variable from "-1" (error) to "0" (success) to avoid "ph_msession_start" wrong error return when video is not supported (and "ph_msession_video_start" method is not called). Please let me know if you have any comments or disagreements about them, ok ? Thanks and best regards, --
|
diff -r dc29b6afa409 wifo/phapi/phapi-old.c
--- a/wifo/phapi/phapi-old.c Mon Jun 15 18:27:14 2009 +0200
+++ b/wifo/phapi/phapi-old.c Tue Jun 16 13:32:49 2009 -0300
@@ -1982,7 +1982,7 @@
if (vl->LineState == LINESTATE_REGISTERED && !skipUnregister)
{
phvlUnregister(ph_vline2vlid(vl));
- vl->LineState == LINESTATE_DELETING;
+ vl->LineState = LINESTATE_DELETING;
}
else
{
diff -r dc29b6afa409 wifo/phapi/phmedia.c
--- a/wifo/phapi/phmedia.c Mon Jun 15 18:27:14 2009 +0200
+++ b/wifo/phapi/phmedia.c Tue Jun 16 13:33:09 2009 -0300
@@ -150,7 +150,7 @@
int ph_msession_start(struct ph_msession_s *s, const char *deviceid)
{
- int ret1,ret2 = -1;
+ int ret1,ret2 = 0;
DBG_MEDIA_ENGINE("MEDIA_ENGINE: entering ph_msession_start\n");
_______________________________________________ QuteCom-dev mailing list [email protected] http://lists.qutecom.org/mailman/listinfo/qutecom-dev
