C / C++ / MFC

int sql_Count = sqlSrvCollection.Count();
    int length = 0;
    for ( int i = 0; i < sql_Count; ++i )
    {
        length += wcslen(sqlSrvCollection[i]);
        length += 2;    // don't forget to add one for ; and the trailing null
    }

    // allocate a buffer big enough to hold the strings
    WCHAR* pzServerNames = new WCHAR[length + 1];    // plus the very final extra null
    PWSTR pNext = pzServerNames;    // a temporary pointer

    // now go through copying each string to the array buffer
    for ( int i = 0; i < sql_Count; ++i )
    {
        wcscpy_s(pNext, length, sqlSrvCollection[i]);                       // copy the next name into the array
        int nlen = wcslen(sqlSrvCollection[i]);                             // get its length
        pNext += nlen;                                                      // move the pointer forward
        *pNext++ = L';';                                                    // add the ; to tokenize
        *pNext++ = L'\0';                                                   // add the null (incrementing the pointer)
        length -= nlen + 1;                                                 // reduce amount of space left
    }
    *pNext = L'\0';    // add the final trailing null

jkirkerx wrote:

...But I'm unsure if I did the right thing.

Quote:

CComBoBox

CString CXbeefixedDlg::getCurStrInCombobox(const CComBoBox &a)

char Singature[16] = { 0 };

sdancer75 wrote:

The CString its a dynamic structure and dont need to make reservations in size before.