PRINCE KUMAR DUBEY | 20 Jun 2012 15:25

Re: [E-devel] enlightenment-devel Digest, Vol 74, Issue 78

Hi Flavio,
I am still not able to reproduce the seg-fault. Is seg-fault happening always ?
Please find my reply to your queries inline below.

Thanks & Regards,
Prince

------- Original Message -------
Date: Mon, 18 Jun 2012 14:09:59 -0300
From: Fl?vio Ceolin 
Subject: Re: [E-devel] [Patch] [Edje] Added: Pulseaudio remix plug-in
for edje multisense module
To: Enlightenment developer list

Message-ID:

Content-Type: text/plain; charset=ISO-8859-1

Hi Prince,

On Mon, Jun 18, 2012 at 9:56 AM, PRINCE KUMAR DUBEY
wrote:
> Hi Flavio,
>
> I tested "edje_player multisense.edje" to reproduce seg fault as mentioned below, but couldn't succeed
with memcheck as well.
> My svn rev is 72374. Please let me know the scenario to reproduce it.
>
> And regarding the point 5, you are right, multisense will not build without remix as per current configure file.
> But, multisense is not all about only sound, it can have vibration etc. So, if we make remix optional for
(Continue reading)

Flavio Ceolin | 20 Jun 2012 20:40

Re: [E-devel] enlightenment-devel Digest, Vol 74, Issue 78

PRINCE KUMAR DUBEY <prince.dubey <at> samsung.com> writes:

Hi Prince,

> Hi Flavio,
> I am still not able to reproduce the seg-fault. Is seg-fault happening
> always ?

Yep, it's happening always i use the edje_player with the multisense
example.

The problem is:

+   length = count * sizeof(RemixCount);

it makes length be greater than PA_PLAYER_BUFFERLEN then:

+        *(player->playbuffer + i) = (PLAYER_PCM) value;

will access invalid memory.

I rewrote this function and worked as expected. Please check if it makes sense.

static void
pa_player_playbuffer(RemixEnv *env __UNUSED__, PA_Player_Data *player, RemixPCM *data, RemixCount count)
{
   int ret;
   RemixCount i, j;
   RemixPCM value;
   size_t length, total_written;
(Continue reading)

PRINCE KUMAR DUBEY | 21 Jun 2012 15:46

Re: [E-devel] enlightenment-devel Digest, Vol 74, Issue 116

Hi,

I would like to set context for new feature haptic/vibration play in multisense.
As we know multisense feature is already in edje which support only audio play.
In this patch, I am adding haptic play support in multisense module.

Regards,
Prince

------- Original Message -------
Date: Thu, 21 Jun 2012 13:04:12 +0000 (GMT)
From: PRINCE KUMAR DUBEY 
Subject: [E-devel] [New Feature] Haptic play support to edje
multisense module
To: "enlightenment-devel <at> lists.sourceforge.net"

Cc: "csmani <at> samsung.com" 
Message-ID: <0M5Y00D6VXNDSDD0 <at> mailout3.samsung.com>
Content-Type: text/plain; charset="windows-1252"

Hi,

Please review the attached patch for new feature haptic/vibration support added to edje multisense module.
This patch includes a sample haptic module which shows the haptic input parameters. This module can be
replaced with an actual haptic interface for haptic effect.
Change Description:
1. New Feature: Haptic play support added to edje multisense module.
2. Multisense support enabled by default.
3. Disabling of Multisense module initialization/shutdown(include remix module initialization which
is required only during runtime) in EDC compilation mode.
(Continue reading)

Gustavo Sverzut Barbieri | 21 Jun 2012 16:19

Re: [E-devel] enlightenment-devel Digest, Vol 74, Issue 116

On Thu, Jun 21, 2012 at 10:46 AM, PRINCE KUMAR DUBEY
<prince.dubey <at> samsung.com> wrote:
> Hi,
>
> I would like to set context for new feature haptic/vibration play in multisense.
> As we know multisense feature is already in edje which support only audio play.
> In this patch, I am adding haptic play support in multisense module.

there was no attachment :-)

Also, don't just patch Edje.  Raster now wants a ecore_audio to
abstract audio from multisense and later be usable from outside it
(I've heard Stefan is working on that). Then it would be nice to have
the ecore_haptics to provide the haptics feedback, and that used by
Edje.

Getting things right from the beginning are way less painful than fix
it later ;-)

--

-- 
Gustavo Sverzut Barbieri
http://profusion.mobi embedded systems
--------------------------------------
MSN: barbieri <at> gmail.com
Skype: gsbarbieri
Mobile: +55 (19) 9225-2202

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
(Continue reading)

Daniel Willmann | 21 Jun 2012 17:12

Re: [E-devel] enlightenment-devel Digest, Vol 74, Issue 116

Hello,

On 06/21/2012 03:19 PM, Gustavo Sverzut Barbieri wrote:
> On Thu, Jun 21, 2012 at 10:46 AM, PRINCE KUMAR DUBEY
> <prince.dubey <at> samsung.com>  wrote:

> Also, don't just patch Edje.  Raster now wants a ecore_audio to
> abstract audio from multisense and later be usable from outside it
> (I've heard Stefan is working on that). Then it would be nice to have
> the ecore_haptics to provide the haptics feedback, and that used by
> Edje.

actually I'm going to be working on ecore_audio. :-)

I'm currently looking at edje_multisense to understand what is needed.
Having haptics support in a separate library rather than inside edje 
makes sense to me as there are probably more users of that API.

> Getting things right from the beginning are way less painful than fix
> it later ;-)

Agreed, though the definition of "right" might evolve over time. :-)

Regards,
Daniel

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
(Continue reading)


Gmane