Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

The window was removing the app_server connection too early, and thus hang in Quit().

Browse Source 
Removed the stopConnection() method.
Cleaned up header a bit.


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@12959 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Axel Dörfler 2005-06-05 18:57:55 +00:00
parent 2e1184a6e4
commit e17b33e56b
2 changed files with 275 additions and 346 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

203
headers/os/interface/Window.h
View File

@ -1,5 +1,5 @@
//------------------------------------------------------------------------------ //------------------------------------------------------------------------------
// Copyright (c) 2001-2002, OpenBeOS // Copyright (c) 2001-2005, Haiku
// //
// Permission is hereby granted, free of charge, to any person obtaining a // Permission is hereby granted, free of charge, to any person obtaining a
// copy of this software and associated documentation files (the "Software"), // copy of this software and associated documentation files (the "Software"),
@ -23,38 +23,20 @@
// Author: Adrian Oanca (adioanca@mymail.ro) // Author: Adrian Oanca (adioanca@mymail.ro)
// Description: BWindow is the base class for all windows (graphic areas // Description: BWindow is the base class for all windows (graphic areas
// displayed on-screen). // displayed on-screen).
//
//
//!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
// NOTE(!!!) All private non virtual functions are to be renamed following
// this scheme: word_another_third <-> wordAnotherThird
// OBOS Team: If you use one of BWindow's private functions, please addapt to this code
//!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
//------------------------------------------------------------------------------ //------------------------------------------------------------------------------
#ifndef _WINDOW_H #ifndef _WINDOW_H
#define _WINDOW_H #define _WINDOW_H
// Standard Includes -----------------------------------------------------------
// System Includes -------------------------------------------------------------
#include <BeBuild.h> #include <BeBuild.h>
#include <InterfaceDefs.h> #include <InterfaceDefs.h>
#include <List.h> #include <List.h>
#include <Looper.h> #include <Looper.h>
#include <Rect.h> #include <Rect.h>
#include <StorageDefs.h> #include <StorageDefs.h>
// Project Includes ------------------------------------------------------------
#include <View.h> #include <View.h>
// Local Includes --------------------------------------------------------------
// Local Defines ---------------------------------------------------------------
class BPortLink; class BPortLink;
// Globals ---------------------------------------------------------------------
// window definitions ---------------------------------------------------------- // window definitions ----------------------------------------------------------
@ -136,17 +118,12 @@ class ViewAttr;
// BWindow class --------------------------------------------------------------- // BWindow class ---------------------------------------------------------------
class BWindow : public BLooper { class BWindow : public BLooper {
public: public:
BWindow(BRect frame, BWindow(BRect frame, const char* title,
const char* title, window_type type, uint32 flags,
window_type type,
uint32 flags,
uint32 workspace = B_CURRENT_WORKSPACE); uint32 workspace = B_CURRENT_WORKSPACE);
BWindow(BRect frame, BWindow(BRect frame, const char* title,
const char* title, window_look look, window_feel feel,
window_look look,
window_feel feel,
uint32 flags, uint32 flags,
uint32 workspace = B_CURRENT_WORKSPACE); uint32 workspace = B_CURRENT_WORKSPACE);
virtual ~BWindow(); virtual ~BWindow();
@ -156,7 +133,7 @@ static BArchivable *Instantiate(BMessage* data);
virtual status_t Archive(BMessage* data, bool deep = true) const; virtual status_t Archive(BMessage* data, bool deep = true) const;
virtual void Quit(); virtual void Quit();
void Close(); // Synonym of Quit() void Close() { Quit(); }
void AddChild(BView* child, BView* before = NULL); void AddChild(BView* child, BView* before = NULL);
bool RemoveChild(BView* child); bool RemoveChild(BView* child);
@ -170,21 +147,16 @@ virtual void WorkspacesChanged(uint32 old_ws, uint32 new_ws);
virtual void WorkspaceActivated(int32 ws, bool state); virtual void WorkspaceActivated(int32 ws, bool state);
virtual void FrameResized(float new_width, float new_height); virtual void FrameResized(float new_width, float new_height);
virtual void Minimize(bool minimize); virtual void Minimize(bool minimize);
virtual void Zoom( BPoint rec_position, virtual void Zoom(BPoint origin, float width, float height);
float rec_width,
float rec_height);
void Zoom(); void Zoom();
void SetZoomLimits(float maxWidth, float maxHeight); // changed from: SetZoomLimits(float max_h, float max_v); void SetZoomLimits(float maxWidth, float maxHeight);
virtual void ScreenChanged(BRect screen_size, color_space depth); virtual void ScreenChanged(BRect screen_size, color_space depth);
void SetPulseRate(bigtime_t rate); void SetPulseRate(bigtime_t rate);
bigtime_t PulseRate() const; bigtime_t PulseRate() const;
void AddShortcut( uint32 key, void AddShortcut(uint32 key, uint32 modifiers,
uint32 modifiers,
BMessage *msg); BMessage *msg);
void AddShortcut( uint32 key, void AddShortcut(uint32 key, uint32 modifiers,
uint32 modifiers, BMessage *msg, BHandler *target);
BMessage* msg,
BHandler* target);
void RemoveShortcut(uint32 key, uint32 modifiers); void RemoveShortcut(uint32 key, uint32 modifiers);
void SetDefaultButton(BButton* button); void SetDefaultButton(BButton* button);
BButton *DefaultButton() const; BButton *DefaultButton() const;
@ -192,11 +164,12 @@ virtual void MenusBeginning();
virtual void MenusEnded(); virtual void MenusEnded();
bool NeedsUpdate() const; bool NeedsUpdate() const;
void UpdateIfNeeded(); void UpdateIfNeeded();
BView *FindView(const char* viewName) const; // changed from: FindView( const char* view_name ); BView *FindView(const char *viewName) const;
BView *FindView(BPoint) const; BView *FindView(BPoint) const;
BView *CurrentFocus() const; BView *CurrentFocus() const;
void Activate(bool = true); void Activate(bool = true);
virtual void WindowActivated(bool state); virtual void WindowActivated(bool state);
void ConvertToScreen(BPoint* pt) const; void ConvertToScreen(BPoint* pt) const;
BPoint ConvertToScreen(BPoint pt) const; BPoint ConvertToScreen(BPoint pt) const;
void ConvertFromScreen(BPoint* pt) const; void ConvertFromScreen(BPoint* pt) const;
@ -205,11 +178,13 @@ virtual void WindowActivated(bool state);
BRect ConvertToScreen(BRect rect) const; BRect ConvertToScreen(BRect rect) const;
void ConvertFromScreen(BRect* rect) const; void ConvertFromScreen(BRect* rect) const;
BRect ConvertFromScreen(BRect rect) const; BRect ConvertFromScreen(BRect rect) const;
void MoveBy(float dx, float dy); void MoveBy(float dx, float dy);
void MoveTo(BPoint); void MoveTo(BPoint);
void MoveTo(float x, float y); void MoveTo(float x, float y);
void ResizeBy(float dx, float dy); void ResizeBy(float dx, float dy);
void ResizeTo(float width, float height); void ResizeTo(float width, float height);
virtual void Show(); virtual void Show();
virtual void Hide(); virtual void Hide();
bool IsHidden() const; bool IsHidden() const;
@ -234,23 +209,16 @@ virtual void Hide();
bool IsActive() const; bool IsActive() const;
void SetKeyMenuBar(BMenuBar* bar); void SetKeyMenuBar(BMenuBar* bar);
BMenuBar *KeyMenuBar() const; BMenuBar *KeyMenuBar() const;
void SetSizeLimits( float minWidth, // changed from: SetSizeLimits(float min_h, float max_h, float min_v, float max_v); void SetSizeLimits(float minWidth, float maxWidth,
float maxWidth, float minHeight, float maxHeight);
float minHeight, void GetSizeLimits(float *minWidth, float *maxWidth,
float maxHeight); float *minHeight, float *maxHeight);
void GetSizeLimits( float *minWidth, // changed from: SetSizeLimits(float* min_h, float* max_h, float* min_v, float* max_v);
float *maxWidth,
float *minHeight,
float *maxHeight);
uint32 Workspaces() const; uint32 Workspaces() const;
void SetWorkspaces(uint32); void SetWorkspaces(uint32);
BView *LastMouseMovedView() const; BView *LastMouseMovedView() const;
virtual BHandler *ResolveSpecifier(BMessage* msg, virtual BHandler *ResolveSpecifier(BMessage* msg, int32 index,
int32 index, BMessage* specifier, int32 form, const char* property);
BMessage* specifier,
int32 form,
const char* property);
virtual status_t GetSupportedSuites(BMessage* data); virtual status_t GetSupportedSuites(BMessage* data);
status_t AddToSubset(BWindow* window); status_t AddToSubset(BWindow* window);
@ -273,31 +241,21 @@ virtual status_t Perform(perform_code d, void* arg);
bool IsModal() const; bool IsModal() const;
bool IsFloating() const; bool IsFloating() const;
status_t SetWindowAlignment(window_alignment mode, status_t SetWindowAlignment(window_alignment mode, int32 h,
int32 h, int32 hOffset = 0, int32 width = 0, int32 widthOffset = 0,
int32 hOffset = 0, int32 v = 0, int32 vOffset = 0, int32 height = 0,
int32 width = 0,
int32 widthOffset = 0,
int32 v = 0,
int32 vOffset = 0,
int32 height = 0,
int32 heightOffset = 0); int32 heightOffset = 0);
status_t GetWindowAlignment(window_alignment* mode = NULL, status_t GetWindowAlignment(window_alignment* mode = NULL,
int32* h = NULL, int32* h = NULL, int32* hOffset = NULL,
int32* hOffset = NULL, int32* width = NULL, int32* widthOffset = NULL,
int32* width = NULL, int32* v = NULL, int32* vOffset = NULL,
int32* widthOffset = NULL, int32* height = NULL, int32* heightOffset = NULL) const;
int32* v = NULL,
int32* vOffset = NULL,
int32* height = NULL,
int32* heightOffset = NULL) const;
virtual bool QuitRequested(); virtual bool QuitRequested();
virtual thread_id Run(); virtual thread_id Run();
// Private or reserved --------------------------------------------------------- // Private or reserved ---------------------------------------------------------
private: private:
typedef BLooper inherited; typedef BLooper inherited;
friend class BApplication; friend class BApplication;
@ -326,16 +284,13 @@ virtual void _ReservedWindow8();
BWindow(BWindow&); BWindow(BWindow&);
BWindow &operator=(BWindow&); BWindow &operator=(BWindow&);
BWindow(BRect frame, color_space depth, // to be implemented BWindow(BRect frame, color_space depth,
uint32 bitmapFlags, int32 rowBytes); uint32 bitmapFlags, int32 rowBytes);
void InitData(BRect frame, void InitData(BRect frame, const char* title,
const char* title, window_look look, window_feel feel,
window_look look, uint32 flags, uint32 workspace);
window_feel feel, status_t ArchiveChildren(BMessage* data, bool deep) const;
uint32 flags, status_t UnarchiveChildren(BMessage* data);
uint32 workspace);
status_t ArchiveChildren(BMessage* data, bool deep) const; // call made from within Archive
status_t UnarchiveChildren(BMessage* data); // Instantiate(BMessage* data)->BWindow(BMessage)->UnarchiveChildren(BMessage* data)
void BitmapClose(); // to be implemented void BitmapClose(); // to be implemented
virtual void task_looper(); // thread function - it's here where app_server messages are received virtual void task_looper(); // thread function - it's here where app_server messages are received
/* void start_drag( BMessage* msg, /* void start_drag( BMessage* msg,
@ -392,11 +347,10 @@ virtual BMessage *ConvertToMessage(void* raw, int32 code);
void SetIsFilePanel(bool yes); void SetIsFilePanel(bool yes);
bool IsFilePanel() const; bool IsFilePanel() const;
// OBOS BWindow's addon functions
uint32 WindowLookToInteger(window_look wl); uint32 WindowLookToInteger(window_look wl);
uint32 WindowFeelToInteger(window_feel wf); uint32 WindowFeelToInteger(window_feel wf);
void BuildTopView(); void BuildTopView();
void stopConnection();
void setFocus(BView *focusView, bool notifyIputServer = false); void setFocus(BView *focusView, bool notifyIputServer = false);
bool handleKeyDown(uint32 key, uint32 modifiers); bool handleKeyDown(uint32 key, uint32 modifiers);
@ -404,7 +358,7 @@ virtual BMessage *ConvertToMessage(void* raw, int32 code);
// message: B_MOUSE_UP, B_MOUSE_DOWN, B_MOUSE_MOVED // message: B_MOUSE_UP, B_MOUSE_DOWN, B_MOUSE_MOVED
void sendMessageUsingEventMask(int32 message, BPoint where); void sendMessageUsingEventMask(int32 message, BPoint where);
BView* sendMessageUsingEventMask2(BView* aView, int32 message, BPoint where); BView* sendMessageUsingEventMask2(BView* aView, int32 message, BPoint where);
void sendPulse( BView* ); void sendPulse(BView *view);
int32 findShortcut(uint32 key, uint32 modifiers); int32 findShortcut(uint32 key, uint32 modifiers);
bool findHandler(BView* start, BHandler* handler); bool findHandler(BView* start, BHandler* handler);
BView* findView(BView* aView, const char* viewName) const; BView* findView(BView* aView, const char* viewName) const;
@ -423,53 +377,53 @@ virtual BMessage *ConvertToMessage(void* raw, int32 code);
// Debug // Debug
void PrintToStream() const; void PrintToStream() const;
// END: OBOS addon functions
// 3 deprecated calls // 3 deprecated calls
//void AddFloater(BWindow* a_floating_window); //void AddFloater(BWindow* a_floating_window);
//void RemoveFloater(BWindow* a_floating_window); //void RemoveFloater(BWindow* a_floating_window);
//window_type WindowType() const; //window_type WindowType() const;
char *fTitle; // used private:
int32 server_token; char *fTitle;
bool fInTransaction; // used // changed from: char fInUpdate; int32 server_token; // not yet used
bool fActive; // used // changed from: char f_active; bool fInTransaction;
short fShowLevel; // used bool fActive;
uint32 fFlags; // used short fShowLevel;
uint32 fFlags;
port_id send_port; // used port_id send_port;
port_id receive_port; // used port_id receive_port;
BView *top_view; // used BView *top_view;
BView *fFocus; // used BView *fFocus;
BView *fLastMouseMovedView; // used BView *fLastMouseMovedView;
void *a_session; // unused //changed from _BSession_ void *a_session; // unusedm was _BSession_
BMenuBar *fKeyMenuBar; // used BMenuBar *fKeyMenuBar;
BButton *fDefaultButton; // used BButton *fDefaultButton;
BList accelList; // used BList accelList;
int32 fTopViewToken; // // changed from: int32 top_view_token; int32 fTopViewToken;
bool fPulseEnabled; // used // changed from: bool pulse_enabled; bool fPulseEnabled;
bool fViewsNeedPulse; bool fViewsNeedPulse; // not yet used
bool fIsFilePanel; // used bool fIsFilePanel;
bool fMaskActivated; bool fMaskActivated;
bigtime_t fPulseRate; // used // changed from: bigtime_t pulse_rate; bigtime_t fPulseRate;
bool fWaitingForMenu; bool fWaitingForMenu;
bool fMinimized; // used // changed from: bool fOffscreen; bool fMinimized;
sem_id fMenuSem; sem_id fMenuSem;
float fMaxZoomHeight; // used // changed from: float fMaxZoomH; float fMaxZoomHeight;
float fMaxZoomWidth; // used // changed from: float fMaxZoomV; float fMaxZoomWidth;
float fMinWindHeight; // used // changed from: float fMinWindH; float fMinWindHeight;
float fMinWindWidth; // used // changed from: float fMinWindV; float fMinWindWidth;
float fMaxWindHeight; // used // changed from: float fMaxWindH; float fMaxWindHeight;
float fMaxWindWidth; // used // changed from: float fMaxWindV; float fMaxWindWidth;
BRect fFrame; // used BRect fFrame;
window_look fLook; // used window_look fLook;
ViewAttr *fCurDrawViewState; ViewAttr *fCurDrawViewState; // not yet used
window_feel fFeel; // used window_feel fFeel;
int32 fLastViewToken; int32 fLastViewToken;
BPortLink *fLink; // used // changed from: _CEventPort_* fEventPort; BPortLink *fLink;
BMessageRunner *fPulseRunner; // used BMessageRunner *fPulseRunner;
BRect fCurrentFrame; BRect fCurrentFrame; // not yet used
uint32 _reserved[2]; // was 8 uint32 _reserved[2]; // was 8
#if !_PR3_COMPATIBLE_ #if !_PR3_COMPATIBLE_
@ -477,19 +431,4 @@ virtual BMessage *ConvertToMessage(void* raw, int32 code);
#endif #endif
}; };
// inline definitions ----------------------------------------------------------
inline void BWindow::Close()
{
Quit();
}
//------------------------------------------------------------------------------
#endif // _WINDOW_H #endif // _WINDOW_H
/*
* $Log $
*
* $Id $
*
*/

18
src/kits/interface/Window.cpp
View File

@ -242,6 +242,10 @@ BWindow::~BWindow()
// disable pulsing // disable pulsing
SetPulseRate(0); SetPulseRate(0);
// tell app_server about our demise
fLink->StartMessage(AS_DELETE_WINDOW);
fLink->Flush();
delete fLink; delete fLink;
delete_port(receive_port); delete_port(receive_port);
} }
@ -328,10 +332,6 @@ BWindow::Quit()
// ... also its children // ... also its children
//detachTopView(); //detachTopView();
STRACE(("Trying to stop connection...\n"));
// tell app_server, this window will finish execution
stopConnection();
STRACE(("Connection stopped!\n"));
if (fFlags & B_QUIT_ON_WINDOW_CLOSE) if (fFlags & B_QUIT_ON_WINDOW_CLOSE)
be_app->PostMessage(B_QUIT_REQUESTED); be_app->PostMessage(B_QUIT_REQUESTED);
@ -2385,16 +2385,6 @@ BWindow::BuildTopView()
} }
void
BWindow::stopConnection()
{
Lock();
fLink->StartMessage(AS_DELETE_WINDOW);
fLink->Flush();
Unlock();
}
void void
BWindow::prepareView(BView *view) BWindow::prepareView(BView *view)
{ {