Opened 14 years ago

Closed 13 years ago

#177 closed bug (fixed)

input_server must be notified about shutdown

Reported by: marcusoverhagen Owned by: bonefish
Priority: normal Milestone: R1
Component: - General Version:
Keywords: Cc: diver
Blocked By: Blocking:
Has a Patch: no Platform: All

Description (last modified by leavengood)

input server must be notified that the system is beeing shut down, as it has to notify BInputServerDevice::SystemShuttingDown()

BeBook:

Tells the object that the Input Server is in the process of shutting down. Unless something interrupts the shutdown, this notification will be followed by a Stop() and delete, thus you don't have to do much from this function (other than note that the end is near).

Change History (8)

comment:1 by marcusoverhagen, 14 years ago

blocked: 180

comment:2 by bonefish, 14 years ago

Status: newassigned

comment:3 by bonefish, 14 years ago

Please suggest a message protocol. And add the message codes to some private header (headers/private/app/RegistrarDefs.h? Or a new header?).

comment:4 by marcusoverhagen, 14 years ago

Perhaps registrar should quit input_server after all other applications have quit.

I don't see a valid reason for keeping input_server running after shutdown.

Having the "Reboot" button after the system has been shut down isn't very important in my opinion.

With APM or ACPI working, the system will power off after shutdown, so the "Reboot" button would only be visible/useful on hardware where both APM and ACPI is broken.

comment:5 by diver, 14 years ago

Cc: diver added

comment:6 by korli, 13 years ago

Ingo, would be it ok to have the registrar send a warning message (SYSTEM_SHUTTING_DOWN) to vital apps instead of ignoring them ?

comment:7 by bonefish, 13 years ago

I suppose instead of a special message we can just send B_QUIT_REQUESTED to the "vital" apps after all other apps have been quit. If you need it now, please feel free to adjust the shutdown code (I've switched my working dirs to my branch at the moment).

comment:8 by leavengood, 13 years ago

Description: modified (diff)
Platform: All
Resolution: fixed
Status: assignedclosed

Fixed as of hrev19289. The input server is now notified of the shutdown.

Note: See TracTickets for help on using tickets.