Age | Message |
---|---|
8 years 5 months | Put code to construct unique path into new utlity function
uniquePath() Two almost identical versions existed of this code. I generalized the code by taking out the knowledge of sections. No other uses of this code are planned, but it is easier to review in isolation now. |
8 years 5 months | Put code for moving links to a different section into Menu class This is another step in bringing the management of the links on the file system into a single location (the Menu class). Fixes were applied as well. For example a directory for the new section is now created if it did not exist yet. |
8 years 5 months | Fix bug in section directory creation method The child directory (the one bearing the section name) was never created; instead the parent directory (named "sections") was created twice. |
8 years 5 months | Make sure inserting a new section doesn't change the selected section |
8 years 5 months | Changed LinkApp::renameFile() to setFile() The original method name suggested it might rename the file on disk, which it does not. |
8 years 5 months | Add "/compile" to .gitignore This file is put there by one of the GNU autotools. |
8 years 5 months | Moved section directory delete code to Menu class It is better to have all the file handling code for menu entry files in the same place. |
8 years 5 months | Only allow empty sections to be deleted Since not all links can be deleted, an action that deletes a section and all the links it contains cannot be reliably implemented. Therefore, the action will only be offered if all links have already been deleted (or were never added in the first place). |
8 years 5 months | Removed "gp2x.h" header None of the definitions in it were used anymore. |
8 years 5 months | Set a window title This is useful when testing on PC. |
8 years 5 months | Declare "dirp" and "dptr" pointers just before they're needed |
8 years 5 months | Cleaned up code that loads a manual from an OPK file Most of it is for readability only, but there is also a leak that was plugged: opk_close() is now called if extraction fails. |
8 years 5 months | Gave Layer class a protected constructor |
8 years 5 months | Removed unused MMSP2ADC struct definition A leftover from platform code that was removed long ago? |
8 years 5 months | Fixed formatting string type mismatches The "%zd" conversion specification seems to produce no warnings on both 32-bit and 64-bit systems. |
8 years 5 months | Removed unused methods in LinkApp class |
8 years 5 months | Suppress Clang analyzer warnings about dead assignments The last assignment in the series is indeed dead, but for consistency I want to keep it around. |
8 years 5 months | Quit GMenu2X when window is closed in main screen This helps when testing on PC. I didn't bother to implement the same functionality for all dialog event loops, since that would be a lot of extra work and not necessary for the scenario that I need it for: clean exit when running under Valgrind. |
8 years 5 months | Removed the "Rename section" option This option is not compatible with section names being provided by other sources than user-created links. Examples of other sources are built-in action links, links installed by a distro in a read-only directory and metadata from OPK files. |
8 years 11 months | Remove all touch-screen-related code It didn't work anymore, at all. Touchscreen polling occurred only *after* waiting for a button had completed. So the touchscreen events, if any, would be ignored until the user had pressed a button, possibly firing off another action with that button press and forwarding the touchscreen event to the next interface that got brought up as a result. Reusing this code and fixing it would require far more work than rewriting everything anew with touchscreen devices in mind from the beginning. MtH: Resolved conflicts, mainly from the GMenu2X pointer to reference change I did on 2015-04-21. |
Commit 2a2634b3640286bf05fa5158e5f71ba1d2490aea,
by Nebuleon Fumika
|
|
8 years 5 months | Removed include of <ini.h> No functions from libini are used, so this line was probably left there by accident. |
8 years 5 months | Keep track of Link objects using unique_ptr This ensures the memory is freed when the pointers are removed. While there was an explicit delete loop in the destructor, memory leaks could occur when links were removed from the menu. |
8 years 5 months | Use const references to strings in Menu method arguments This avoids unnecessary copying. Removed sectionName argument to addLink(), since its only caller didn't provide a value for it, meaning it always defaulted. |
8 years 5 months | Keep list of section names sorted at all times Keeping it sorted when sections are inserted after startup improves the user experience and if we have code for that already, we might as well use it during startup too. |
8 years 5 months | Create sections for built-in action links if necessary Previously, the built-in actions would only be added if their respective sections already existed. This also works around the fact that Menu::paint() crashes if there are no sections present. |
Branches:
install_locations
master
opkrun
packages