Reorganisation #29
Reorganisation #29
Conversation
…link instead of hardcode
…senders, implementors)
StevenCostiou
left a comment
StevenCostiou
left a comment
There was a problem hiding this comment.
J'ai quelques doutes sur certaines suppressions, et peut-être qu'il y a des méthodes de DSToolInfo supprimées mais toujours appelées.
| DSMouseEnterWindowRecord >> record: morphsAndWindow [ | ||
|
|
||
| | window tool | | ||
| super record: morphsAndWindow. | ||
| window := morphsAndWindow last. | ||
| window ifNil: [ ^ self ]. | ||
| tool := DSSpy toolInWindow: window. | ||
| tool ifNil: [ ^ self ]. | ||
| toolInfo := DSToolInfo | ||
| toolNamed: tool class name | ||
| id: tool identityHash | ||
| windowId: windowId | ||
| action: 'enter' | ||
| ] | ||
|
|
||
| { #category : 'accessing' } | ||
| DSMouseEnterWindowRecord >> toolInfo [ |
There was a problem hiding this comment.
J'ai un doute sur cette suppression, c'est une entrée fenêtre, et on enregistre un tool info, qui est le moyen dans l'history et les outils de repérer dans quel outil on se trouve.
There was a problem hiding this comment.
C'est toujours présent mais dans la classe mère DSMouseEventRecord, ce qui permet d'unifier les records d'entrée et sortie de fenêtres.
On enregistre toujours le toolInfo (très important en effet) mais il reste des bugs où il se met Nil alors qu'il ne devrait pas... (typiquement quand on close une fenêtre ça fait un "CloseWindow" et un "LeaveWindow" en même temps et ça se gère mal pour le moment)
| self buildWindowHistory. | ||
| self buildWindows. | ||
| self buildWindowJumps. | ||
| "self buildWindowJumps." |
There was a problem hiding this comment.
Le build des window jumps serait à améliorer, tu l'as commenté à cause d'un pb ?
There was a problem hiding this comment.
Oui j'ai dû commenter des trucs dans le build de l'history car c'était bloquant en fonction de ce que tu record
|
|
||
| ^ toolAction | ||
| ] | ||
|
|
There was a problem hiding this comment.
Idem sur cette suppression, pour moi c'était important. Mais peut-être que c'est pris en compte et comme je lis séquentiellement le diff, je ne vois pas encore comment.
| toolInfo := DSToolInfo | ||
| toolNamed: tool class name | ||
| id: tool identityHash | ||
| windowId: windowId |
There was a problem hiding this comment.
j'ai l'impression que cette méthode de classe a été supprimée dans DSToolInfo
There was a problem hiding this comment.
Non, je viens de vérifier elle est toujours là
…or instrumentation and add a parameter to inform about debugging sessions
…strumentation already record the action
Julesboul
force-pushed
the
Reorganisation
branch
from
78949bb to
4a9593c
Compare
2 participants
Replace hardcode instrumentation by Metalinks and announcement listeners.
In addition, we clean the code by deleting some instrumentations (which are not needed anymore).