For now the code assumes a 6630 but in order to support other
> devices we will probably need to add a run-time check on the device
> id so this works on non-OMAP based devices.

Does this mean this code will not run on phones like the 3650? We still
need to support older Series60 devices on the 150Cay branch.


+#if (0)
 static TInt FlagToNumber(TMMFSampleRate flag)
 {
     switch( flag )
@@ -186,9 +179,10 @@
     default:
         break;
     }
-    HX_ASSERT(false);
+    HX_ASSERT(FALSE);
     return 0;
 }
+#endif

Are we going to use this #if 0 code later?

+        static const TInt KClientPriority = 69;
+
     // set priority.
     TMMFPrioritySettings prioritySettings;
-    prioritySettings.iPref     = KPriorityPref;
+        prioritySettings.iPref     = EMdaPriorityPreferenceTime;
     prioritySettings.iPriority = KClientPriority;
     prioritySettings.iState    = EMMFStatePlaying;
     m_pStream->SetPrioritySettings(prioritySettings);
+    }

What is priority 69 and is this changing from the old one?

rest looks good.

--greg.

_______________________________________________
Audio-dev mailing list
[email protected]
http://lists.helixcommunity.org/mailman/listinfo/audio-dev

Reply via email to