Открыли, типа. Помнится, тут тусуются знатоки раскошерного C++, вопрос к ним. Каков код? Нравится?

Здравствуйте, smeeld, Вы писали:

S>Открыли, типа. Помнится, тут тусуются знатоки раскошерного C++, вопрос к ним. Каков код? Нравится?

При беглом просмотре по виду классический код от MS. Кучи дефайнов, процедур. Простейшие ОО-обёртки над процедурами. Почти никаких тебе темплейтов. STL, по-моему, вообще не используется (но это не точно).
Объявления классов в исходных кодах, а не хидерах, считаю некошерно. В исходниках не присутствует ни одного комментария.

S>Открыли, типа. Помнится, тут тусуются знатоки раскошерного C++, вопрос к ним. Каков код? Нравится?
глянул пару файлов. для ~25летнего проекта код неплох

Здравствуйте, Maniacal, Вы писали:

M>Объявления классов в исходных кодах, а не хидерах, считаю некошерно. В исходниках не присутствует ни одного комментария.

Комментарии там выпилены специально. А вот частое отсутствие дефолтной 0инициализации локальный переменных в функциях-это по меркам любителей раскошерного C++ просто убийственно.

Здравствуйте, ononim, Вы писали:

S>>Открыли, типа. Помнится, тут тусуются знатоки раскошерного C++, вопрос к ним. Каков код? Нравится?
O>глянул пару файлов. для ~25летнего проекта код неплох

Этот код всюду юзается в продуктивах по всему миру. В отличчии от...

Здравствуйте, smeeld, Вы писали:

S>Открыли, типа. Помнится, тут тусуются знатоки раскошерного C++, вопрос к ним. Каков код? Нравится?

Хорошо, что не нужно такое сопровождать. Этот фрагмент:

                const WCHAR*    rgpsz[ 5 ];
                DWORD           irgpsz      = 0;
                WCHAR           szAbsPath[ IFileSystemAPI::cchPathMax ];
                WCHAR           szOffset[ 64 ];
                WCHAR           szLength[ 64 ];
                WCHAR           szFailures[ 64 ];
                WCHAR           szElapsed[ 64 ];

                CallS( pfmp->Pfapi()->ErrPath( szAbsPath ) );
                OSStrCbFormatW( szOffset, sizeof( szOffset ), L"%I64i (0x%016I64x)", OffsetOfPgno( pgnoStart ), OffsetOfPgno( pgnoStart ) );
                OSStrCbFormatW( szLength, sizeof( szLength ), L"%u (0x%08x)", ( pgnoEnd - pgnoStart + 1 ) * g_cbPage, ( pgnoEnd - pgnoStart + 1 ) * g_cbPage );
                OSStrCbFormatW( szFailures, sizeof( szFailures ), L"%i", iRetry );
                OSStrCbFormatW( szElapsed, sizeof( szElapsed ), L"%g", ( TickOSTimeCurrent() - tickStart ) / 1000.0 );

                rgpsz[ irgpsz++ ]   = szAbsPath;
                rgpsz[ irgpsz++ ]   = szOffset;
                rgpsz[ irgpsz++ ]   = szLength;
                rgpsz[ irgpsz++ ]   = szFailures;
                rgpsz[ irgpsz++ ]   = szElapsed;

                UtilReportEvent(    eventError,
                                    LOGGING_RECOVERY_CATEGORY,
                                    TRANSIENT_READ_ERROR_DETECTED_ID,
                                    irgpsz,
                                    rgpsz,
                                    0,
                                    NULL,
                                    m_pinst );

вообще вызвал слезы умиления, без такой инициализации rgpsz при помощи irgpsz ну никак нельзя было.

Здравствуйте, smeeld, Вы писали:

S>Этот код всюду юзается в продуктивах по всему миру.
Это не работает как аргумент в пользу того, что сейчас надо писать также.

Здравствуйте, smeeld, Вы писали:

S>Открыли, типа. Помнится, тут тусуются знатоки раскошерного C++, вопрос к ним. Каков код? Нравится?
Тут уже отметили, что коду 25 лет. Для своего времени — норма, сейчас писать можно лучше.

Здравствуйте, Skorodum, Вы писали:

S>Тут уже отметили, что коду 25 лет. Для своего времени — норма, сейчас писать можно лучше.

Там кучи приколов, которые нельзя списать на неразвитость стандартов, или компиляторов, или чего ещё, так как просто тупистика, с точки зрения современного раскошерного C++-ника. Хотя, если ту самую раскошерность считать разновидностью современного смузихлёбства, то тупистикой становится уже современная С++-ная раскошерность, а не те приколы в том коде.

S>Там кучи приколов, которые нельзя списать на неразвитость стандартов
Каких приколов? Сильно копаться лень, а пока ничего экстраординарного не заметил, обычный код до-stl-ной эпохи.

S>Открыли, типа. Помнится, тут тусуются знатоки раскошерного C++, вопрос к ним. Каков код? Нравится?
Кстати, а почему такая спешка? Выложили несобираемый код, т.к. без билдскриптов. Коменты просто выпилили безо всякого ревью. Пишут что со временем все допилят, но вопрос — зачем было в спешке выкладывать в паблик недопиленное для паблика?