FENÊTRE DE DEBUG GTK+ POUR OPENGL

Commentaire de Arnaud16022 le 24/12/2006 17:38:06

Oh juste une remarque en passant : J'ai fait cette source pour pouvoir modifier rapidement des petits trucs chiants genre la couleur de particules, etc, sans avoir à compiler 20 fois pour trouver la bonne teinte ou la bonne taille ou une taille de police qui va bien, etc.
C'est uniquement à ça que ça sert ;)

Joyeuses fêtes.

Commentaire de Funto66 le 31/12/2006 01:18:57

Yep salut Arnaud (et Kirua au passage ^^), je viens de parcourir ta source :)

Je vais faire ce que je n'ai pas fait depuis si longtemps sur CppFrance : une critique constructive :D
Donc d'abord les points positifs : très très bonne source en fait, c'est vrai que c'est très pratique et propre, simple d'utilisation, utile, bien commentée et "personnalisable" avec quelques #defines : j'aime beaucoup :)

Bon maintenant les points négatifs :p :
-l'indentation où tu mélanges des espaces et des tabulations...'fin ça c'est personnel hein mais j'aime bien quand c'est homogène...bon ok on s'en fout :p
-#define CGTKwin dumpGTKwin ou #define CGTKwin _CGTKwin : pour quoi faire?
Pourquoi ne pas faire plus simplement :

#ifndef GTKWIN_DISABLE
class CGTKwin
{
void Init(); // l'implémentation sera dans le .cpp
// blabla...
};
#else
class CGTKwin
{
void Init(){} // implémentation vide
// blabla...
};
#endif
-pourquoi get_win() et par GetWin() ? Ta façon de nommer les fonctions n'est pas homogène...
-les variagles globales qui sont dans le namespace NDebug : soit les mettre en static au sens C du terme, c'est à dire non visibles depuis un autre fichier .cpp (ça ne coûte rien et c'est plus propre ^^), soit les mettre en private et static au sens C++ du terme dans la classe CGTKwin, ce qui est bcp + logique non??
-on peut écrire sur la zone de texte GTK+, je pense qu'on doit pouvoir empêcher ça en 1 seule ligne de code...tu me diras, de toutes façons c'est juste pour du débuggage donc c'est pas important...
-au démarrage de ton prog, le focus est donné à la fenêtre GTK+, alors que ce serait plus logique de le donner à la fenêtre GLFW. Détail aussi, et puis ça dépend surtout du programme que l'on débuggue plutôt que de CGTKwin : peut-on créer une fenêtre GTK+ sans qu'elle ait le focus?? (ou sinon faudrait que CGTKwin appelle une fonction GLFW qui donnerait le focus à la fenêtre GLFW : cette fonction existe-t-elle?)

Bon et puis quelques fautes d'ortho/de frappe...(dsl ^^) :
-appellés -> appelés (CGTKwin.cpp)
-appeller -> appeler (CGTKwin.cpp)
-Fenêtre désactivés -> Fenêtre désactivée (CGTKwin.h)
-calbacks -> callbacks (CGTKwin.cpp)

Voili voilou, ça faisait longtemps que je ne faisais pas ça :)

A tous les programmeurs OpenGL : c'est vraiment utile, vous devriez vous en servir, et puis c'est un pote à moi qui l'a faite, elle est forcément bien cette source :p