История изменений

Исправление Obey-Kun, 12.06.13 04:25 (текущая версия) :

~~Dron~~, что за точки с запятой после закрывающих фигурных скобок? Плюс ты что, про code conventions не слышал? Мне вот эти нравятся: http://techbase.kde.org/Policies/Kdelibs_Coding_Style.

Реализуется вот так:

astyle --indent=spaces=4 --brackets=linux \
       --indent-labels --pad-oper --unpad-paren --pad-header \
       --keep-one-line-statements --convert-tabs --indent-preprocessor \
       --preserve-date --formatted --suffix=none --add-brackets --align-pointer=name \
       `find src -type f -name '*.cpp' -or -name '*.cc' -or -name '*.h' -or -name '*.h.in'`

Так читается лучше:

if (point_init_all == OFF) {
    result = unien_read_conf(str_conf, "init_video");
    if (strcmp("yes", result) == 0) {
        unien_init_video();
    }

    result = unien_read_conf(str_conf, "init_audio");
    if (strcmp("yes", result) == 0) {
        unien_init_audio();
    }

    result = unien_read_conf(str_conf, "init_timer");
    if (strcmp("yes", result) == 0) {
        unien_init_timer();
    }

    result = unien_read_conf(str_conf, "init_cdrom");
    if (strcmp("yes", result) == 0) {
        unien_init_cdrom();
    }

    result = unien_read_conf(str_conf, "init_joystick");
    if (strcmp("yes", result) == 0) {
        unien_init_joystick();
    }

    result = unien_read_conf(str_conf, "init_eventthread");
    if (strcmp("yes", result) == 0) {
        unien_init_eventthread();
    }

    result = unien_read_conf(str_conf, "init_opengl");
    if (strcmp("yes", result) == 0) {
        unien_init_opengl();
    }
}

Ну и вообще говоря, лучше уж тогда вот так:

bool unien_enabled_in_conf(char *option, char *str_conf)
{
    char *conf_val = unien_read_conf(str_conf, option);
    return strcmp("yes", conf_val) == 0;
}

if (point_init_all == OFF) {
    if (unien_enabled_in_conf("init_video", str_conf)) {
        unien_init_video();
    }
    if (unien_enabled_in_conf("init_audio", str_conf)) {
        unien_init_audio();
    }
    if (unien_enabled_in_conf("init_timer", str_conf)) {
        unien_init_timer();
    }
    if (unien_enabled_in_conf("init_cdrom", str_conf)) {
        unien_init_cdrom();
    }
    if (unien_enabled_in_conf("init_joystick", str_conf)) {
        unien_init_joystick();
    }
    if (unien_enabled_in_conf("init_eventthread", str_conf)) {
        unien_init_eventthread();
    }
    if (unien_enabled_in_conf("init_opengl", str_conf)) {
        unien_init_opengl();
    }
}

Исходная версия Obey-Kun, 12.06.13 04:23:

if (point_init_all == OFF) {
    result = unien_read_conf(str_conf, "init_video");
    if (strcmp("yes", result) == 0) {
        unien_init_video();
    }

    result = unien_read_conf(str_conf, "init_audio");
    if (strcmp("yes", result) == 0) {
        unien_init_audio();
    }

    result = unien_read_conf(str_conf, "init_timer");
    if (strcmp("yes", result) == 0) {
        unien_init_timer();
    }

    result = unien_read_conf(str_conf, "init_cdrom");
    if (strcmp("yes", result) == 0) {
        unien_init_cdrom();
    }

    result = unien_read_conf(str_conf, "init_joystick");
    if (strcmp("yes", result) == 0) {
        unien_init_joystick();
    }

    result = unien_read_conf(str_conf, "init_eventthread");
    if (strcmp("yes", result) == 0) {
        unien_init_eventthread();
    }

    result = unien_read_conf(str_conf, "init_opengl");
    if (strcmp("yes", result) == 0) {
        unien_init_opengl();
    }
}

Ну и вообще говоря, лучше уж тогда вот так:

bool unien_enabled_in_conf(char *option, char *str_conf)
{
    char *conf_val = unien_read_conf(str_conf, option);
    return strcmp("yes", conf_val) == 0;
}

if (point_init_all == OFF) {
    if (unien_enabled_in_conf("init_video", str_conf)) {
        unien_init_video();
    }
    if (unien_enabled_in_conf("init_audio", str_conf)) {
        unien_init_audio();
    }
    if (unien_enabled_in_conf("init_timer", str_conf)) {
        unien_init_timer();
    }
    if (unien_enabled_in_conf("init_cdrom", str_conf)) {
        unien_init_cdrom();
    }
    if (unien_enabled_in_conf("init_joystick", str_conf)) {
        unien_init_joystick();
    }
    if (unien_enabled_in_conf("init_eventthread", str_conf)) {
        unien_init_eventthread();
    }
    if (unien_enabled_in_conf("init_opengl", str_conf)) {
        unien_init_opengl();
    }
}