Discussion :

[robux]

Bonjour,

J'ai une appli sous XP qui marche mais sous vista elle plante. En pas à pas, je supçonne un CreateWindow() qui retourne un mauvais Handle.

J'ai pu voir sur google certaines choses sur ce sujet mais ça me semble pas assez complet.

quelqu'un aurait-il rencontré ce problème? Est ce qu'il faut un flags ou un argument spécial à changer pour cette fonction sous vista?

Voici mon code :

Une classe Notification crée une window et la call back recoit les messages. En pas à pas le hwnd dans la call back est mauvais. il y a une adresse de pointeur mais il y a marqué "unsed=???" après :

mHwnd = 0x001606d8 {unused=??? }
unused = CXX0030: Error: expression cannot be evaluated

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
LCNotificationWindow::LCNotificationWindow(HINSTANCE aHInstance, LPCWSTR aAppName, bool aIsStatusItem, 
										   int aWidthWindow, int aHeightWindow)
{
	mHInstance = aHInstance;
	mIsStatusItem = aIsStatusItem;

	WNDCLASS lWC;
	lWC.style = 0;
    lWC.lpfnWndProc = WndProc;
    lWC.cbClsExtra = 0;
    lWC.cbWndExtra = 0;
    lWC.hInstance = mHInstance;
	lWC.hIcon = LoadIcon(mHInstance, MAKEINTRESOURCE("IDI_MAINICON"));
    lWC.hCursor = LoadCursor(mHInstance, IDC_ARROW);
    lWC.hbrBackground = (HBRUSH)(1 + COLOR_BTNFACE);
    lWC.lpszMenuName =  NULL;
    lWC.lpszClassName = _T("LCNotificationWindowClass");

	RegisterClass(&lWC);
	mHwnd = CreateWindow(_T("LCNotificationWindowClass"), aAppName, WS_OVERLAPPEDWINDOW,
						CW_USEDEFAULT, CW_USEDEFAULT, aWidthWindow, aHeightWindow,
						NULL, NULL, mHInstance, NULL);
	int lol = GetLastError();
	SetWindowLongPtr(mHwnd, GWLP_USERDATA, (LONG)(LONG_PTR)this);

}

Voici la call back, j'arrive dans WM_COMMAND et le hwnd (qui vient du createwindow est mauvais) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17

LRESULT CALLBACK LCNotificationWindow::WndProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
	LCNotificationWindow* lNotificationWindowPt;
	switch (uMsg)
	{
		...

		case WM_COMMAND:
			lNotificationWindowPt = (LCNotificationWindow*)(LONG_PTR)GetWindowLongPtr(hwnd, GWLP_USERDATA);
			lNotificationWindowPt->onNotificationCommand(hwnd, uMsg, wParam, lParam);
			return 0;

		...	
	}
}

Veuillez noter que le GetLastError retourne 0 après le CreateWindow()

Merci

Robux

[Médinoc]

Un HWND n'est pas supposé être un pointeur valide.
Il est déclaré en tant que pointeur uniquement pour profiter d'un typage pas-trop-faible en C.

Par contre, ceci est à bannir sur tout système 64 bits:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

SetWindowLongPtr(mHwnd, GWLP_USERDATA, (LONG)(LONG_PTR)this);

Si tu as des warnings quand tu compiles en 32 bits, va voir ceci à la place.

PS: Je conseillerais plutôt de faire le SetWindowLongPtr dans le traitement de WM_CREATE (voire WM_NCCREATE), aussi. Regarde l'aide du message pour voir comment récupérer des infos en paramètre.

[robux]

Merci de tes réponses Medinoc :

Un HWND n'est pas supposé être un pointeur valide.
Il est déclaré en tant que pointeur uniquement pour profiter d'un typage pas-trop-faible en C..

Certes mais que me proposes tu? Veux tu dire que ce qui suit est normal??? je ne comprend pas ta réponse

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
mHwnd = 0x001606d8 {unused=??? }
unused = CXX0030: Error: expression cannot be evaluated

Par contre, ceci est à bannir sur tout système 64 bits:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

SetWindowLongPtr(mHwnd, GWLP_USERDATA, (LONG)(LONG_PTR)this);

Si tu as des warnings quand tu compiles en 32 bits, va voir ceci à la place.

PS: Je conseillerais plutôt de faire le SetWindowLongPtr dans le traitement de WM_CREATE (voire WM_NCCREATE), aussi. Regarde l'aide du message pour voir comment récupérer des infos en paramètre.

Merci je vais tester sous vista 64 et voir ce que ça donne. Cependant le prototype de la fonction est le suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
...

#define SetWindowLongPtr  SetWindowLongPtrW

...

#define SetWindowLongPtrW   SetWindowLongW

...

SetWindowLongW(
    __in HWND hWnd,
    __in int nIndex,
    __in LONG dwNewLong);

Autrement dit la fonction demande un LONG en 3eme paramètre donc le cast en long est plus que necéssaire. Non?


Robux

[Médinoc]

Certes mais que me proposes tu? Veux tu dire que ce qui suit est normal??? je ne comprend pas ta réponse
<snip>

Oui, c'est normal.

Merci je vais tester sous vista 64 et voir ce que ça donne. Cependant le prototype de la fonction est le suivant :
<snip>
Autrement dit la fonction demande un LONG en 3eme paramètre donc le cast en long est plus que necéssaire. Non?

Le prototype est ainsi quand tu compiles en 32 bits. Quand tu compiles en 64 bits, SetWindowLongPtrW() est une fonction à part entière, et non une macro sur SetWindowLongW().

Le code dont je t'ai donné le lien ne fait rien quand tu compiles en 64 bits: Il se contente de modifier les macros en 32 bits pour que tu n'aies pas de warning fallacieux.

[robux]

C'est ok pour le cast je t'ai donné le mauvais prototype de la fonction il y en a un autre pour w64 qui prend un LONG_PTR. Le cast en LONG n'est pas necessaire.


Après recherche le problème ne vient pas du createwindow mais d'un SetMenu.

-> Très important : Sous vista le Setmenu ne peut pas se faire après un CreatePopupMenu. Il se fait après un CreateMenu.

Conséquence : Je ne passe plus par le hWnd pour avoir le menu mais je le stocke directement en membre. Et ça marche.

Merci

Robux