SHS/shstk
4
1
Fork 0
Code Issues Pull Requests Releases Wiki Activity

pip, qad #103

Merged
peri4 merged 4 commits from master into release 2021-08-07 11:59:43 +03:00
Conversation 3 Commits 4 Files Changed 2 +2 -2
peri4 commented 2021-08-01 22:12:44 +03:00
Owner
Copy Link

ждем @andrey из отпуска, погляди и принимай

ждем @andrey из отпуска, погляди и принимай
peri4 added 1 commit 2021-08-01 22:12:46 +03:00
pip, qad 69d871433f
andrey self-assigned this 2021-08-02 08:57:36 +03:00
andrey commented 2021-08-02 09:02:00 +03:00
Owner
Copy Link

Всё супер, но есть пара замечаний:
Надо бы переписать пооптимальнее вот это место
а то нехорошо понапарасному вызывать конструктор, а PIVector вообще можно из void * сделать напрямую из памяти, и также поправить тут

Всё супер, но есть пара замечаний: Надо бы переписать пооптимальнее вот это [место](https://git.shs.tools/SHS/qad/src/commit/c0ac0002e6159c3933f1ad43d78ec7e33915396b/libs/piqt/piqt.h#L125-L128) а то нехорошо понапарасному вызывать конструктор, а PIVector вообще можно из void * сделать напрямую из [памяти](https://git.shs.tools/SHS/qad/src/commit/c0ac0002e6159c3933f1ad43d78ec7e33915396b/libs/piqt/piqt.h#L194), и также поправить [тут](https://git.shs.tools/SHS/qad/src/commit/c0ac0002e6159c3933f1ad43d78ec7e33915396b/libs/piqt/piqt.h#L195-L200)
andrey commented 2021-08-02 09:26:39 +03:00
Owner
Copy Link

И кстати надо убрать из сервисов dpkg удаление пользователя, так нельзя делать.

И кстати надо убрать из сервисов dpkg удаление пользователя, так [нельзя](https://unix.stackexchange.com/questions/47880/how-debian-package-should-create-user-accounts) делать.
peri4 commented 2021-08-03 00:06:02 +03:00
Author
Owner
Copy Link

сервисы поправил, один метод оптимизировал. В сторону Qt так нельзя, у них нет такого конструктора. Глянь и принимай

сервисы поправил, один метод [оптимизировал](https://git.shs.tools/SHS/qad/commit/fdd8c0c637743b6f4fcad29d93884f61df00ffb6). В сторону Qt так нельзя, у них нет такого конструктора. Глянь и принимай
peri4 added 1 commit 2021-08-03 05:58:23 +03:00
pip, qad 900f2874ac
peri4 added 1 commit 2021-08-03 21:35:36 +03:00
qad 725dd09336
peri4 added 1 commit 2021-08-07 11:59:15 +03:00
pip,qad 98eea66a49
peri4 merged commit a0b4ae652c into release 2021-08-07 11:59:43 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Баг
Косяк
 В работе
Ставим как приступили к выполнению
 Назначена
Ставим когда назначен исполнитель
 Отложено
Выполнение задачи отложено или ожидает других
 Сделано
Ставим когда выполнили задачу, и ожидаем одобрения
 Тестируется
Ставим когда отправили задачу на тестирование
 Требует обсуждения
Задача вызывает трудности или вопросы
 Фича
Новый функционал
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
andrey drone jenkins peri4
No Assignees andrey
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SHS/shstk#103
Delete Branch "master"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?