На своих-то, но такие ошибки могут стоить недели дебага

ну я примерно тоже самое сказал, но хозяин - барин

умение предугадыавть ошибки и подстелить сена - это исключительно опыт

так же как и обратная сторона, понимание где подстилать не нужно

я конечно пока что полностью этого лишен

@anatolijs дай, пожалуйста, пример кода, как бы ты защитил такой кусок

size_t outSize; uint8_t *buffer; mrb_dump_irep(mrb, irep, 0, &buffer, &outSize); char *p = (char *)buffer; char_vector_t out(p, p + outSize); mrb_free(mrb, buffer); mrb_close(mrb); return out;

вот это было реально полезным финальным аккордом в обсуждении

умение предугадыавть ошибки и подстелить сена - это исключительно опыт

Тогда выучи и повторяй перед сном Valgrid наш, на светится memchecker твой, да прибудет vm твоя

а mrb как определен?

mrb_state*

size_t outSize; uint8_t *buffer; mrb_dump_irep(mrb, irep, 0, &buffer, &outSize); char *p = (char *)buffer; char_vector_t out(p, p + outSize); mrb_free(mrb, buffer); mrb_close(mrb); return out;

Я так понимаю, mrb_dump_irep возвращает код ошибки, а не исключение выбрасывает

да, это так

Я так понимаю, mrb_dump_irep возвращает код ошибки, а не исключение выбрасывает

какие исключени в си то

но Анатолий же сейчас говорит о НЕПРЕДВИДЕННОМ

ну так главное тут обработать код ошибки mrb_dump_irep

Если ему не удалось выделить память, то дальше при создании вектора ( не знаю, что за тип) может быть исключение

еще если mrb открывается вне функции, то и закрывать его надо вне функции, не надо такого рода менеджмент вносить внутрь обработчика

еще если mrb открывается вне функции, то и закрывать его надо вне функции, не надо такого рода менеджмент вносить внутрь обработчика

нет, таких глупостей я не делаю

он открывается в функции

черт, мне нужен экстеншн вс для гиста, щас поставлю

поставил

https://gist.github.com/micktu/946460a2a30ce74df57d1d85767356ca

вот вся функция

ну, хотя бы на коленках defer: size_t outSize = 0; uint8_t *buffer = nullptr; mrb_state* mrb = nullptr; mrb_dump_irep(mrb, irep, 0, &buffer, &outSize); std::unique_ptr < mrb_state, std::function<void(mrb_state*)» holder(mrb, [](mrb_state*mrb) { mrb_close(mrb); }); std::unique_ptr < uint8_t, std::function<void(uint8_t*)» holder(buffer, [&mbr](uint8_t *buffer) { mrb_free(mrb, buffer); }); char *p = (char *)buffer; char_vector_t out(p, p + outSize);

сколько боли

вы привыкли к этой бесчеловечности?

ну, во всяком случае все твои динамические ресурсы защищены

по хорошему надо было просто обертку на mbr_state написать и все

но ты же спешишь

да кстати я и не заметил что у меня указатель баффер не инициализирова

по хорошему надо было просто обертку на mbr_state написать и все

я напишу просто обертку над функциями загрузки и дампа скрипта

я напишу просто обертку над функциями загрузки и дампа скрипта

чтобы бизнес-логика не занималась открытиями и закрытиями мруби-контекста

там можно уже и ошибочки обработать, и плохие указатели поймать

std::unique_ptr < uint8_t, std::function<void(uint8_t*)>> holder(buffer, [&mbr](uint8_t *buffer) { mrb_free(mrb, buffer); }); за это спасибо, научился

вообще не умею еще с++-указатели, не пользовался. знаю о них и не более

std::unique_ptr < uint8_t, std::function<void(uint8_t*)>> holder(buffer, [&mbr](uint8_t *buffer) { mrb_free(mrb, buffer); }); за это спасибо, научился

реально хороший паттерн

это стандарт?

но defer стандарными средствами :)

но действительно многолословный

щас будет реальн тупой вопрос

[&mbr] что значит этот синтаксис

ничего не понял!

захват контекта лямбдой

а, блин, это лямбда

господи!

и передача туда ссылки на объект mbr

да, все очевидно

тупак

окей, юник поинтер с декструкторм нравится

возьму на вооружение

масштабирования не вижу, но нужно побороть смещение https://gist.github.com/nib952051/df30e6fea374f54d83768d48031e805d

еще по чисто красивостям вопрос

если я хочу по-простому распарсить и трансформировать строку, что шустрее всего - просто пройти по символам в строке записывая каждый в стрингстрим?

кстати я сказал слово трансформировать и дошло что можно написать трансформ

это куда проще

спасибо

(на самом деле мне нужны функции котораые заменяет \r?\n на буквальное "\n" и наоборот)

(регекс ооочччень медленный)

не очень по теме чата, понимаю

это совсем осноовы :)

просто был вопрос как оптимально работать с такими сиутациями и не упустил ли я какой-то магии которая лучше чем копирование в стрингстрим посимвольно

ваще стрингстрим медленный, вроде, если память не проблема, можно просто аллоцировать массив 2хдлины оригинала и копировать в него

так и поступлю

красота

что может быть лучше простых алгоритмов

правда лучше не рефренс а значение в ренже

А если заимплементировать в функциональном стиле?

А еще лучше конст реф)

А если заимплементировать в функциональном стиле?

вперед :)

состязание

я предпочитаю декларативный потому что он быстрый

даже в других языках

А еще лучше конст реф)

для чара зачем реф и тем более конст реф?

str_t это элиас на string?

А еще лучше конст реф)

ты об аргументе? да там должен быть конст реф

а я говорил что надо не char &c а char c

чуток побыстрее будет если компилятор не догадается оптимизировать

(инб4 догадается)

str_t это элиас на string?

угу, это стд::стринг

Оки.