За что мне это?

[metaclass]

Бездны крестиков
boost_1_49_0\libs\filesystem\v3\src\operations.cpp
https://gist.github.com/3735989

После рекурсивного вызова не проверяется ec, соответственно, если он сломается - ошибка будет возвращена от попытки create_directory, а не настоящая ошибка от рекурсивного вызова.
Надо бы поискать в баг-трекере или отрепортить это дело.
PS: В 1.51.0 баг починен.

Comments

[dair-spb.livejournal.com]

Я уже после названия первой директории бы дальше не ходил.

[nivanych.livejournal.com]

> Бездны крестиков

Бездны крестиковой оккупации!

[levgem.livejournal.com]

какой-то кошмар. Это тоже С++?

[metaclass.livejournal.com]

Да, популярная С++ библиотека :)

[nivanych.livejournal.com]

;-) Ну не эрланг же!

[nivanych.livejournal.com]

Надо уметь превозмогать свой страх и отвращение!
Если есть хоть какое-то обоснование, зачем, конечно ;-)

[trueblacker.livejournal.com]

нда :(
а ещё boost называется!
впрочем, версия уже не свежая

[metaclass.livejournal.com]

В 1.51.0 все починено, судя по виду.

[vaddimka.livejournal.com]

меня как-то достала постоянная модификация интерфейса этой конкретной библиотеки и я ее практически отовсюду выпилил
а так, вообще, сам буст - очень даже гут
но вот если ковырнуть что-то типа asio, там граблей просто тонны

[zamotivator.livejournal.com]

Что значит "не проверяется"? Там есть is_directory - вот если этот метод сломается, ты получишь ошибку сразу.

[zamotivator.livejournal.com]

Забавно, что никто из комментаторов этого не заметил.

[metaclass.livejournal.com]

Что я получу после ошибки в рекурсивном вызове?
Конкретно сейчас я получаю вместо ERROR_ACCESS_DENIED - ERROR_PATH_NOT_FOUND, потому что рекурсивный вызов валится с первой ошибкой, ничего не проверяется и выполняется попытка вызвать create_directory, после чего возвращается уже другая ошибка.

И таки в 1.51.0 это переделано корректно.

[zamotivator.livejournal.com]

 if (p.empty() || exists(p))
    {
      if (!p.empty() && !is_directory(p))

Это что? Это не отваливается? Почему?
Путь должен быть exists и каждый его элемент должен быть директорией. Вот эта штука и свалится.

[metaclass.livejournal.com]

Я про:
// First create branch, by calling ourself recursively
create_directories(p.parent_path(), ec);
// Now that parent's path exists, create the directory
create_directory(p, ec);


тут пропущено что-то вроде if(ec) return false;
но в новой версии они более сложным образом сделали, используя локальный error_code

[zamotivator.livejournal.com]

Тьфу, теперь понял.

Да, это баг, следует починить.

[zamotivator.livejournal.com]

А, т.е. в новой версии починено. Тем лучше :)

[guamoka.livejournal.com]

Э-э. Это такая крестики-спесифик бага? :-)