mainwindow.h
138:142: redesign brief description
! setLastProcess() & setPreviousProcess() - unclear logic
suggest to use container to manage sub-processes
161: Describe in details why it's 1101
162: please re-name it
174: fix align
189: delete it
190: rename to linkageProcess
191-192: Details please
236: details
242: details; when does it happen?
265: correct comments
mainwindow.cpp
67: Ini files should have it's own version system. QSAGUI should be able
to fix incorrect versions
65: refactor ini-files handle
83-121: make them inline
123: remove it
148: MEGA-HACK function; refactor logic
165: think about it; this "if" not necessary :-)
339: Incorrect function name or logic; define stuff about Progress Bar
in another place
391: Set default case: "Unexpected error"
Save "current session" stuff not in the temp dir, but in user/.qsagui
Delete temp files every time when user close GUI
Delete qsagui... dir in the temp directory
Place SA binaries names to variables and give user the possibility to
change them (change default values)
DO NOT MERGE CHANNELS
If sub-process failed GUI output SA err message and return code
Disable possibility to run sub-processes when another sub-process is
performed (e.g. set buttons inactive and so on)
582: launchAndDraw --> runAndShow()
Make all 1-2 lines functions inline
funclistwidget.h
98:rename to SEARCH_BUTTON_SIDE_SIZE
116: childTypeBox --> relationTypeBox
136: Ch+Par = relations (e.g. getCurrRelationsFuncs())
funcslistwidget.cpp
35: delete line
43: rename to "Search for"
Convert scr files to UTF-8, use UTF-8 inside the app
54,112: tr() missed
82: fix line length
96,105: save file name in variable.
99,108: save separator in variable.
168: declare this line as inline func.
specify universal function to run SA binaries (e.g. use it on 179 line
and other similar lines)
--
Best Regards,
Fedor Korshunov
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"SA_team" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.ru/group/sa_team?hl=en
-~----------~----~----~----~------~----~------~--~---
