Opened 11 years ago

Closed 11 years ago

#352 closed enhancement (fixed)

Ugly constat in QtNode

Reported by: mrusu Owned by: mrusu
Priority: trivial Milestone:
Component: qlparser Version: 8.4
Keywords: Cc:
Complexity: Medium

Description

In qtnode.cc there is a constant int QtNodes which holds the number of elements in QtInheritance. Considering that this constant is widely used and new operators are being added all the time, it is very inefficient and a usual source of runtime errors if it is off by 1. There is also no logging associated with it that would help to identify it as a cause in due time. I think it's time to change it to a more flexible approach.

Change History (2)

comment:1 by Dimitar Misev, 11 years ago

I submitted a patch.

comment:2 by mrusu, 11 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.