А вобще мне кажется мы обсуждаем прям какой-то комплекс человека если к нему обратились не так как ОН хотел (ты вместо вы). Других объяснений у меня нет.

+100

Важнее как будет сказана та или иная фраза, нежели употребил ли ты "ВЫ" или "ТЫ". Мой выбор это употребление "ТЫ". Чем мне и нравится английский язык )))

да, ее нужно снести поскорее

Я за QDoc

Да, нужно

Можно как-то так

Ок

Ну я бы неназвал это engine. Скорее это приватные данные приватного класса ))

Можно сделать в этом случае вот как:

C++ (Qt)
class SerialPortPrivateData;
 
class SerialPortPrivate
{
public:
    // some common methods
...
private:
    // some common data
    SerialPortPrivateData *m_data;
};

В SerialPortPrivateData будут "сложены" все переменные класа и методы каждой платформы в отдельности.

И каждый платформенный cpp будет иметь свое объявление и реализацию SerialPortPrivateData.

В таком случае иы измежим нагромождение дефайнов с хэдере.

Путаницы и сложности отладки я ненаблюдал, но снова таки дело вкуса.

Совершенно необоснованный ничем камень в мою сторону. Я так понимаю b-s-a понял очем речь. При одинаковых именах отпадает надобность в методе create() и впринципе в наследовани. Но это всего другой подход к решению одной и тойже задачи.

ЗЫ: А исходники я конечно смотрел.

Вот короткая иллюстрация:

serialport_p.h

C++ (Qt)
class SerialPortPrivate
{
public:
    // some common methods
...
#ifdef Q_WS_WIN
   //some windows specified methods
#endif
#ifdef Q_WS_MAC
   //some mac specified methods
#endif
#ifdef Q_WS_X11
  //some unix specified methods
#endif
.... 
private:
    // some common data
};

В  файле serialport.cpp будет находится общая для всех платформ реализация методов.
В фалах serialport_win.cpp, serialport_mac.cpp, serialport_unix.cpp будет находиться реализация этих методов для каждой платформы

Ну и наконец

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

создавал бы все что нужно вне зависимости от платформы.

В качестве примера можно рассмотреть qthread класс.

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

так зачем наследование и метод create()?


upd: Заглянул в исходники. Несовсем так. Имена классов для различных платформ не одинаковое.

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

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

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