String Buffer sizing

[JasonLpz]

Im using a LB_GETTEXTLEN to get the length on of the string but how do i make the buffer ?

{ADDED}
char *sBuf = "";
int iList;
int iSel;
int iSel2;

iSel = ::SendMessage(list,LB_GETCURSEL,0,0);
iSel2 = ::SendMessage(list,LB_GETTEXTLEN,iSel,0);

::SendMessage(list,LB_GETTEXT,iSel,sBuf);

// iList = ::GetWindowTextLength(list);
// ::GetWindowText(list,sBuf,iList);

::SetWindowText(edit,sBuf);

[jim mcnamara]

use either:

new

or calloc() (malloc() in disguise)

after using the buffer, call either delete or free(). delete goes with new, free() with calloc().

[JasonLpz]

Forgot to say ima newB how do i do it? can i get a sample. Ill research on msdn but can u gimme a sample please thanks

[JasonLpz]

Look at the attached youll see my error

http://www.vbforums.com/attachment.php?s=&postid=1335196

[Chris]

{ADDED}
char *sBuf = NULL;
int iList;
int iSel;
int iSel2;

iSel = ::SendMessage(list,LB_GETCURSEL,0,0);
iSel2 = ::SendMessage(list,LB_GETTEXTLEN,iSel,0);

// Allocate & Initialize the string buffer
sBuf = (char*)LocalAlloc(LPTR, iSel2+1);
memset(sBuf, '\0', iSel2+1);

::SendMessage(list, LB_GETTEXT, iSel, (LPARAM)sBuf);

// iList = ::GetWindowTextLength(list);
// ::GetWindowText(list,sBuf,iList);

::SetWindowText(edit,sBuf);

// Free the used resource
LocalFree((char*)sBuf);



You can use others function to allocate the string buffer like mention by jim mcnamara.

[JasonLpz]

Thats the error i get with your code:

Error C2137: Empty character constant

ON:
memset(sBuf, '', iSel2+1);

[Chris]

The sample code is filered by the PHP tag,

it should be


memset(sBuf, '\0', iSel2+1);

[JasonLpz]

YES Great thanks

if i had a million dollars.......










..........I'd be rich! lol

[CornedBee]

Learn C++.

[JasonLpz]

are you talking to me? Cuz if you are you must not be very smart, because what do you think im trying to do?

[CornedBee]

You are using MFC without knowing enough C++ for it. That's what you're trying to do, and it's a bad thing.

[made_of_asp]

Isn't LocalAlloc depreciated science Windows 3.1?

Use HeapAlloc() instead.

[parksie]

Deprecated ;) Depreciated means losing its value...in this case partially correct though.

{Local,Global}Alloc were designed more around 16-bit programming, yes.

[CornedBee]

Yep, those two were built around the special memory model of Winx.x. I'm happy I didn't program back then.

[Chris]

:p, i should change my code to use HeapAlloc from now.

[CornedBee]

The only one of those function that should still be used is LocalFree, because FormatMessage allocates with LocalAlloc if requested to allocate memory.