Ревизия текущего состояния библиотеки:
1. Считаю, что методы SerialPortPrivate::setReadNotificationEnabled и пр. следует перенести в AbstractSerialPortPrivate. В том числе и нотификатор, дабы не плодить сущностей.
2. Считаю, что SerialPort не должен вызывать приватные/защищенные методы SerialPortPrivate/AbstractSerialPortPrivate (начал работать с этим, но лучше пусть автор сделает)
3. serialport_p.h, думаю, стоит разделить по архитектурам. Не нравятся мне такие нагромождения.

Работоспособность библиотеки пока еще не проверял, но под Linux компилировал.
Все мои изменения в MR.

Нет, нифига при таком подходе не получается.
ИМХО, все-таки нужно было делать через промежуточный класс engine

2 b-s-a
Также глянул твой цикл где ты применяешь бинарный поиск и заметил, что если скорость будет равна 4000000 бод, то она в
список не добавится, т.к. её адрес равен standardRates_end.
Нужно наверное в массив еще добавить любое значение больше чем 4000000 - и тогда должно работать. Не?

С нотификатором хрень получается. Хотя.. хз..

Не, переопределять
скорее всего не придется, т.к. у этих методов одна и таже реализация.


ЗЫ:
И глянь выше мое сообшение (я отредактировал) про бинарный поиск.

>Именно что "не". Смотри, если у тебя массив из N элементов, то что лежит по индексу N (ведь *(array + N) - это аналог array[N])?
Да, точно, по индексу N переменная будет за пределом (N-1) массива лежать и в ней будет всякая хрень. Тупил ночью  

Но с архитектурой приватных классов библиотеки срочно нужно что-то предпринять/переделать. А то какая то хрень получилась.
Но вот что - хз.

ХЗ какая, но я вот предлагаю теперь такую, с учетом твоих пожеланий о разделении по платформам.
Я приаттачил простенький проектик с наброском новой архитектуры.

Там изменения коснулись только приватных классов SerialPortPrivate и SerialPortNotifier.
Теперь классы SerialPortPrivate и  SerialPortNotifier являются абстрактными, и имеют статический метод create(),
который создает наследников SerialPortPrivate и SerialPortNotifier в зависимости от платформы.

т.е. наследниками являются WinSerialPortPrivate, UnixSerialPortPrivate  и т.п., WinSerialPortNotifier , UnixSerialPortNotifier и т.п.
и, соответственно, теперь в конструкторе SerialPort вместо

C++ (Qt)
SerialPort::SerialPort(QObject *parent)
    : QIODevice(parent)
    , d_ptr(new SerialPortPrivate(this))
{}
 

теперь

C++ (Qt)
SerialPort::SerialPort(QObject *parent)
    : QObject(parent)
    , d_ptr(SerialPortPrivate::create(this))
{
}
 

В общем, зацени наметаным глазом, если будет время (желательно побыстрее).

А почему не выбрали для всех архитектур одно имя? Зачем так сложно: абстрактный класс, наследование, статический метод? API данных класов я так понимаю одинаковое. Разместить объявление в одном общем *.h и разнести определение по *.cpp для каждой платформы: *_win.cpp, *_unix.cpp, *_mac.cpp, etc.

Да, с этим согласен. Общие данные нужно вынести конечно

2 b-s-a: А эти данные общие для SerialPortPrivate и SerialPortNotifier или у этих классов свои данные?

так оно и было (есть)

Именно.

Ну так что, годится новая структура?