Создаю библиотеку для работы с последовательными портами. [УШЕЛ ИЗ ПРОЕКТА].

[b-s-a]

Т.е. если приникает в базовую реализацию base.cpp - то и фик с ним?

Ты не путай, "базовую реализацию" с "реализацией общей части". Это две большие разницы, как говорят в Одессе. 

[kuzulis]

ОК,  парни, спасибо,  убедили.

Ща буду объединять твою, b-s-a, часть и то что уже есть на гиториосе.
Вечером (по москве) заккомичу.

[kuzulis]

Всё, закоммитил.

Класс SerialPortInfo реализовал 1:1 как QPinterInfo с небольшими отличиями и, по-моему, получилось неплохо.
В классах SerialPort тоже некоторые отличия: микс кучи всего .

В общем, зацените.

ЗЫ: и на будущее, давайте условимся в выражениях сравнения - константы или функции возвращающие значения писать слева, а переменные справа.

Код

C++ (Qt)
...
    int value = 0;
    if (SerialPort::NoParity == value) {
    ...
    }
...
 

на всякий случай.

[pastor]

ЗЫ: и на будущее, давайте условимся в выражениях сравнения - константы или функции возвращающие значения писать слева, а переменные справа.

Почему так? Мы же всетаки читаем слева-направо, да и сравниваем значение переменной (функции) с неким значением, а не наоборот. Как-то такой вариант мне неочень

[kuzulis]

Чтоб не ошибиться случайно

[b-s-a]

ЗЫ: и на будущее, давайте условимся в выражениях сравнения - константы или функции возвращающие значения писать слева, а переменные справа.

А вот это зря. Если вдруг будем мержить с Qt, то придется судорожно везде исправлять. Тролли не принимают подобный код.

[lit-uriy]

>>Чтоб не ошибиться случайно
А в чём можно ошибиться?

>>Мы же всетаки читаем слева-направо
согласен, это более естественно

[kuzulis]

Ох, как всё сложно то... 

[b-s-a]

Ох, как всё сложно то... 

да ладно. вот если бы ты об этом узнал, когда весь код написал, вот тогда да - сложно.

[kuzulis]

Ну ладно.
Ну так что там с структурой классов SerialPort + SerialPortInfo и их приватных подклассов: примерно годится такой вариант или нет?
А то уже руки чешутся начинать реализацию писать

[b-s-a]

Внес ряд изменений:
https://gitorious.org/~bsa/qserialdevice/bsas-qserialdevice/commits/2.0

[kuzulis]

Вопросы:

по SerialPortInfo private.

1. Зачем убрал nullSerialPortInfoPrivate?
Чисто для получения метода isNull() ?

2. Может быть, код:

Код

C++ (Qt)
foreach(const SerialPortInfo &info, availablePorts()) {
    if (port.portName() == info.portName()) {
        *this = info;
	break;
    }
}
 

заменить на:

Код

C++ (Qt)
QList<SerialPortInfo> list = availablePorts();
foreach(const SerialPortInfo &info, list) {
    if (port.portName() == info.portName()) {
        *this = info;
	break;
    }
}
 

?
т.к. по идее нужно всего один раз вызвать availablePorts() , а не каждый раз при итерации.

--

Левые вопросы:
1. После того как я совершу мерж, как мне на сайте gitorious его закрыть?
Просто выбрать статус Closed и написать комментарий?

т.к. как кроме статусов Opened/Closed там больше нет никаких, а тов. lit-uriy говорил, что есть еще и Merged, если я правильно его понял.

[b-s-a]

1. Зачем убрал nullSerialPortInfoPrivate?
Чисто для получения метода isNull() ?

По сути да. Зачем держать в памяти "пустой" объект, когда можно ограничиться нулевым указателем?

2. Может быть, код:

Код

C++ (Qt)
foreach(const SerialPortInfo &info, availablePorts()) {
    if (port.portName() == info.portName()) {
        *this = info;
	break;
    }
}
 

заменить на:

Код

C++ (Qt)
QList<SerialPortInfo> list = availablePorts();
foreach(const SerialPortInfo &info, list) {
    if (port.portName() == info.portName()) {
        *this = info;
	break;
    }
}
 

?
т.к. по идее нужно всего один раз вызвать availablePorts() , а не каждый раз при итерации.

foreach() гарантирует однократное выполнение второго параметра - т.е. availablePorts будет вызван 1 раз.

Левые вопросы:
1. После того как я совершу мерж, как мне на сайте gitorious его закрыть?
Просто выбрать статус Closed и написать комментарий?

т.к. как кроме статусов Opened/Closed там больше нет никаких, а тов. lit-uriy говорил, что есть еще и Merged, если я правильно его понял.

По хорошему, нужно выбирать Merged. Именно такой статус ставят тролли.

[kuzulis]

Ок, насчет foreach не знал.

По хорошему, нужно выбирать Merged. Именно такой статус ставят тролли.

Ну я в упор там (на сайте) не вижу этого статуса: там только можно выбрать из Open/Closed в чекбоксе.
Где взять Merged?
Может это браузер тупит?

[b-s-a]

Подозреваю, что там должна быть кнопка "вмержить".
Создал мерж реквест.