Abdelrazak Younes <[EMAIL PROTECTED]> writes: | > The dispatch function called is not the same dispatch function | > anymore. There might be a reason why we have decided to go through the | > top most dispatch and let that handle the traversement of the dispatch | > hierarchy. | | And what is that reason? I thought it was just a remnant of some old | code practice.
This code, the dispatch hierarchy is not very old. And you have to talk to the creators of this "modern" dispatch thing to get the reason, if any. | > If this is an ivalid reason or not I cannot say, but there | > you have it.... | | I think it is invalid yes. If the traversal could lead to a different | LyXText then I think we are in serious trouble. If there someone | knowing more than that about that stuff. It is not a different lyxtext that is the problem, but a different dispatch. With this change the dispatch hierarchy is entered not at the top anymore. | > the reason why I am not confident that changes like | > this albeit seemingly correct, are wanted. | | Code that is not understandable is bad and should be simplified. I can agree with that, but in general it cannot be simplified before we understand it. -- Lgb