microtrigger / lightpack

Automatically exported from code.google.com/p/lightpack
GNU General Public License v3.0
1 stars 0 forks source link

Изменения структуры файлов проекта #145

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
В последних ревизиях (ветка threads) было 
решено переименовать все файлы проекта 
согласно стилю кодирования (wiki:CodingStyle). В 
основном были переименованы сущности 
(файлы содержащие определения классов) для 
согласования имени опеределяемой сущности 
и имени файла.

Класс MainWindow был переименован в SettingsWindow, для 
улучшения взаимосвязи имени с 
выполняемыми задачами. 
Класс MoveMeWidget переименован в GrabWidget.
QColorButton в ColorButton.

Весь исходный код был перемещен в папку 
Software/src/ в том числе папки grab и alienfx. В этом же 
каталоге создан подпроект для сборки 
основного приложения: src.pro. Т.е. если вам 
нужно собрать Лайтпак, то в настройках Qt 
Creator нужно изменить конфигурацию сборки / 
запуска "src".

Для удобства тестирования работы API были 
написаны тесты каждой команды. Для тестов в 
проекте так же создана отдельная папка 
Software/tests/ в которой размещен подпроект tests.pro 
(он служит роль промежуточной проектной 
папки). Для тестирования API создан проект 
LightpackApiTest.pro в папке tests, в котором применена 
библиотека QTestLib. Для запуска тестов 
достаточно в настройках сборки / запуска Qt 
Creator установить "Запуск: LightpackApiTest". Тесты у 
меня в Windows 7 на данный момент не работают, 
по причине проблем с qtcreator_process_stub.exe, все 
проверки данных тестов производились в Ubuntu 
10.04.

Еще из менее интересных изменений, можно 
упомянуть проблему с файлом Xlib.h. Для 
исключения #include Xlib.h в заголовочном файле 
X11Grabber.hpp была создана приватная структура, 
поля которой определяются в cpp-файлике. 
Проблема с Xlib.h заключается в том что 
какой-то не очень хороший человек написал в 
нем буквально следующее:
#define Status int
Что вызвало конфликт с одним из наших новых 
перечислений (Backlight::Status). В общем-то ничего 
страшного, просто чтобы было понятно 
почему сделано именно так.

Ну в общем такие дела.

Original issue reported on code.google.com by brunql on 14 Sep 2011 at 8:53

GoogleCodeExporter commented 9 years ago

Original comment by tim.hell...@gmail.com on 10 Dec 2012 at 12:41