SourceComment: BItem hierarchy - Removing needless code…

Dear all

While reading BDecoder code, I came upon this class. I think we are storing redundant type info here….

Here is a patch that removes a lot of code

Let me know..

Btw, I generated the patch thru bzr… This is a superb tool,  thanks Peter for introducing me to it… I am going to try the vimdiff plugin today some time… ;-)

—cheers atul

Attached Files:

Would you like to post a relpy?


This post starts a thread.
Follow-ups:
Re: SourceComment: BItem hierarchy - Removing needless code…
Hi Atul, you are perfectly right about this refactoring! If we go a step further we can get rid of type() altogether and use the RTTI instead. For example: if (more...)