9
0
Fork 0

Fix NxWM bug: old window being minimized when new window open

git-svn-id: https://nuttx.svn.sourceforge.net/svnroot/nuttx/trunk@4757 7fd9a85b-ad96-42d3-883c-3090e2eb8679
This commit is contained in:
patacongo 2012-05-21 13:34:09 +00:00
parent 829e68fe76
commit a8cacc56ba
5 changed files with 26 additions and 14 deletions

View File

@ -136,3 +136,7 @@
* NxWM::CTaskbar: After drawing the task bar, need to raise the
application window otherwise the taskbar will be on the top and
keyboard input will not be received by the top application.
* NxWM::CTaskbar: Bugfix... previous window should now be minimized
when a new window is started. It should stay in a maximized state
so that it will re-appear with the window above it is closed or
minimized.

View File

@ -4,7 +4,7 @@ NxWidgets
NxWM
----
(3) General issues
(4) General issues
(3) NxConsole issues
(1) Platform specific issues
@ -55,6 +55,19 @@ o General NxWM Issues
Status: Open
Priority: Low
Title: COMBINE CTouchscreen AND CKeyboard THREADS
Description: Right now, two separate threads handle touchscreen and keyboard/
console input. Each just waits on read() and when toushcscreen
or keyboard input is received, it injects the data into NX.
These two threads should be combined into one thread that waits
on poll for read data from either device. Then when read data
becomes ready for either device, it could perform the read and
data inject from a single thread.
Status: Open
Priority: Low, this is not a product but a text example. If NxWM were
to be productized, this change should be done in order to reduce
stack memory consumption.
o NxConsole Issues
----------------

View File

@ -36,11 +36,6 @@
/********************************************************************************************
* Included Files
********************************************************************************************/
#include <nuttx/config.h> // REMOVE ME
#define CONFIG_DEBUG 1 // REMOVE ME
#define CONFIG_DEBUG_VERBOSE 1 // REMOVE ME
#define CONFIG_DEBUG_GRAPHICS 1 // REMOVE ME
#include <debug.h> // REMOVE ME
#include <nuttx/config.h>
@ -203,7 +198,7 @@ FAR void *CKeyboard::listener(FAR void *arg)
uint8_t rxbuffer[CONFIG_NXWM_KEYBOARD_BUFSIZE];
ssize_t nbytes = read(This->m_kbdFd, rxbuffer, CONFIG_NXWM_KEYBOARD_BUFSIZE);
dbg("nbytes=%d\n"); // REMOVE ME
// Check for errors
if (nbytes < 0)

View File

@ -506,8 +506,7 @@ bool CTaskbar::startApplication(IApplication *app, bool minimized)
return false;
}
// Has the window manager been started? Or were we ask to start
// the application minimized?
// Were we ask to start the application minimized?
if (minimized)
{
@ -551,11 +550,8 @@ bool CTaskbar::topApplication(IApplication *app)
if (m_topApp)
{
// Yes.. then minimize the application
m_topApp->setMinimized(true);
// And make the application non-visible
// Yes.. make the application non-visible (it is still maximized and
// will reappear when the window above it is minimized or closed).
hideApplicationWindow(m_topApp);
}

View File

@ -121,14 +121,18 @@ endif
sort > $(TOPDIR)/System.map
# This is part of the top-level export target
# Note that there may not be a head object if layout is handled
# by the linker configuration.
export_head: board/libboard$(LIBEXT) $(HEAD_OBJ)
ifneq ($(HEAD_OBJ),)
@if [ -d "$(EXPORT_DIR)/startup" ]; then \
cp -f $(HEAD_OBJ) "$(EXPORT_DIR)/startup"; \
else \
echo "$(EXPORT_DIR)/startup does not exist"; \
exit 1; \
fi
endif
# Dependencies