«

Hi
I want to ask how to convert from long to short since i have some
parameter problem with it?
The problem is like below :

short *speech; // Holds length of any given sample

* speech = pMediaSample->GetActualDataLength();

va_g729a_encoder(short *speech, unsigned char *bitstream );

The Calling GetActualDataLength() is in the long declaration and i have
to change it
to short since calling the speech parameter is in short.
The encoder calling part is default and cannot change the short speech
parameter.
So how should resolve it .
Thanks for the help.

"LaMoRt" <> wrote
> I want to ask how to convert from long to short since i have some
> parameter problem with it?
> The problem is like below :
>
> short *speech; // Holds length of any given sample
>
> * speech = pMediaSample->GetActualDataLength();
>
> va_g729a_encoder(short *speech, unsigned char *bitstream );
>
> The Calling GetActualDataLength() is in the long declaration and i have
> to change it
> to short since calling the speech parameter is in short.
> The encoder calling part is default and cannot change the short speech
> parameter.
> So how should resolve it .
> Thanks for the help.
>
To convert a long to a short, simply write

short st;
long lg = 3;

st = (short) lg;

However i suspect that you don't understand how to use pointers. Why is
"speech" a pointer, and what does it point to? Why is "speech" not a short,
if it holds the sample length.

I am not familiar with your call, but I also doubt that the sample length
(nmuber of samples) would be stored in a short - why limit it to 16 bits?
The samples themselves, however, might well be shorts. CDs and many other
systems use 16-bit samples.

LaMoRt wrote:
> Hi
> I want to ask how to convert from long to short since i have some
> parameter problem with it?
> The problem is like below :
>
> short *speech; // Holds length of any given sample

This is a pointer that has not been initialised. It may point to a
random location in memory. It is undefined behaviour to try to
dereference a pointer before it is set to point to a valid location in
memory.

> * speech = pMediaSample->GetActualDataLength();

Here, you attempt to dereference the uninitialised pointer 'speech',
which is undefined behaviour.

> va_g729a_encoder(short *speech, unsigned char *bitstream );
>
> The Calling GetActualDataLength() is in the long declaration and i have
> to change it
> to short since calling the speech parameter is in short.
> The encoder calling part is default and cannot change the short speech
> parameter.
> So how should resolve it .
> Thanks for the help.

How about something like:

short speech;
speech = pMediaSample->GetActualDataLength();
va_g729a_encoder(&speech, bitstream);

--
Simon.

Thanks for the reply.
Actually I am not really good at pointer and it is in 16 bit beacuse
the G.729 codec need it in 16 bit.Actually i am trying to integrate to
our encoder so we just have to call the function that are available on
the API since it have a library that we could include.
The short *speech is pointed to another header file.
* speech = pMediaSample->GetActualDataLength();
i want to pass the actual data length to speech and use it as a
parameter to another function which is
va_g729a_encoder(short *speech, unsigned char *bitstream );

the GetActualDataLength is initialize as long, and we need to follow
the va_g729a_encoder function as it is an API and the speech parameter
is in short.
so how should i convert long to short?

In article <. com>,
"LaMoRt" <> wrote:

> Hi
> I want to ask how to convert from long to short since i have some
> parameter problem with it?
> The problem is like below :
>
> short *speech; // Holds length of any given sample
>
> * speech = pMediaSample->GetActualDataLength();
>
> va_g729a_encoder(short *speech, unsigned char *bitstream );
>
> The Calling GetActualDataLength() is in the long declaration and i have
> to change it
> to short since calling the speech parameter is in short.
> The encoder calling part is default and cannot change the short speech
> parameter.
> So how should resolve it .
> Thanks for the help.

If I were you, I would very very _very_ carefully check the
documentation of the function va_g729a_encoder. Since the number of
bytes needed to store 0.2 seconds of CD quality sound is already too
large to be stored into a short portably, there seems to be something
wrong here.

"Malcolm" <> writes:
[...]
> To convert a long to a short, simply write
>
> short st;
> long lg = 3;
>
> st = (short) lg;

Or, even more simply:

short st;
long lg = 3;

st = lg;

--
Keith Thompson (The_Other_Keith) kst- <http://www.ghoti.net/~kst>
San Diego Supercomputer Center <*> <http://users.sdsc.edu/~kst>
We must do something. This is something. Therefore, we must do this.

In article <>, Keith Thompson <kst-
> writes
>"Malcolm" <> writes:
>[...]
>> To convert a long to a short, simply write
>>
>> short st;
>> long lg = 3;
>>
>> st = (short) lg;
>
>Or, even more simply:
>
>short st;
>long lg = 3;
>
>st = lg;
>

Thanks....

I was just updating the Arianne 6 navigation system....


--
\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\
\/\/\/\/\ Chris Hills Staffs England /\/\/\/\/
/\/\/ www.phaedsys.org \/\/\
\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/\/

Thanks all , i got it already ...
I will check it properly for the documents.
U all give me a good help, thanks so much.