Ошибки, баги, вопросы - страница 3437

[A100]

Vladimir Simakov #:

Это всего лишь пример для доказательства наличия проблемы - без лишних подробностей, а memcpy вообще взята для простоты

Если для Вас принципиально - замените пример на такой:

#import "msvcrt.dll"
    long memcpy( string&, string&, long );
#import
void OnStart()
{
    string  text = "ABCDEFG";
    memcpy( text, text, 0 );
}

результат не изменится:  1 leaked strings left

[Vladimir Simakov]

A100 #:

Это всего лишь пример для доказательства проблемы без лишних подробностей, а memcpy вообще взята для простоты

Если для Вас принципиально - замените пример на такой:

результат не изменится

И опять 2 UB))) Overlaping objects и Non-trivial copyable objects

[Vladimir Simakov]

A100 #:

Это всего лишь пример для доказательства наличия проблемы - без лишних подробностей, а memcpy вообще взята для простоты

Если для Вас принципиально - замените пример на такой:

результат не изменится:  1 leaked strings left

И еще. string - это некий объект. ИМХО конечно, но там, что-то типа указателя на память в куче и размера строки.

[A100]

Vladimir Simakov #:

И еще. string - это некий объект. ИМХО конечно, но там, что-то типа указателя на память в куче и размера строки.

Если у Вас будут не предположения, а доказательства, то сообщите

[A100]

Вообще убрал memсpy из примера, чтобы оппоненты на него не ссылались:

//c.cpp
void f( void* ) {}

//c.mq5
#import "c.dll" 
    void f( string& );
#import
void OnStart()
{
    string text = "ABCDEFG";
    f( text );
}

Результат тот же: 1 leaked strings left

[fxsaber]

A100 #:
Вообще убрал memcpy из примера, чтобы оппоненты на него не ссылались: Результат тот же: 1 leaked strings left

Среди файлов Include\WinAPI\*.mqh можно в ME поискать строку "string &". После чего попробовать официальные (от MQ) найденные сигнатуры на наличие бага.

[A100]

fxsaber #:

Среди файлов Include\WinAPI\*.mqh можно в ME поискать строку "string &". После чего попробовать официальные (от MQ) найденные сигнатуры на наличие бага.

Я уже по максимуму упростил, а Вы предлагаете усложнить - тут простейшие выкладки пользователи не понимают

Кроме того заглядывал как то в  Include\WinAPI\ там сразу 2 ошибки в описании обнаружил

[fxsaber]

A100 #:

предлагаете усложнить

Предполагал, что "string &" не найду, т.к. везде маячит "const string". Думал, что MQ в курсе особенности. Но увидел, что сами MQ "string &" используют в WinAPI. Поэтому возникло предложение.

[A100]

fxsaber #:

Предполагал, что "string &" не найду, т.к. везде маячит "const string". Думал, что MQ в курсе особенности. Но увидел, что сами MQ "string &" используют в WinAPI. Поэтому возникло предложение.

Предложение хорошее, если бы не одно НО:

Понадобилась мне как то функция GetDlgItemText. Нахожу я ее в  Include\WinAPI\

uint GetDlgItemTextW(HANDLE dlg,int nIDDlgItem,string str,int max);

А она не работает. Долго (!) ищу у себя ошибку, не могу понять в чем дело, а в итоге оказалось, что она не работает только потому, что неправильно записана. Ну ладно, все бывает, все ошибаются. Пишу здесь на сайте об этой ошибке

И что Вы думаете - сейчас посмотрел: Include\WinAPI\winuser.mqh - а все без изменений - как она была лет 5 назад неправильно записана, так и до сих пор все

Так что если исправят утечку памяти - хорошо, а нет - так я уже нашел как обойти это

[Maxim Kuznetsov]

A100 #:
Вообще убрал memсpy из примера, чтобы оппоненты на него не ссылались: Результат тот же: 1 leaked strings left

& убери

то есть или const string & или просто string

// C/C++
#include <string.h>
#include <wchar.h>

#define MQL_API(x) __declspec(dllexport) x __stdcall

MQL_API(long) bred(wchar_t *w)
{
        return wcslen(w);
}

компилим

gcc -o bred.dll -shared bred.c 

используем

#import "bred.dll"
long bred(string s);
#import

void OnStart()
{
   string w="ABCDEF";
   long len=bred::bred(w);
   PrintFormat("length %s = %ld",w,len);
}

и никаких вам leaked strings :-)