[decompiler] Better support for non-virtual methods

[water111]

Previously the decompiler didn't care if a method call was virtual or not. Now it checks to see if it is what it expects, and gives an error. By default, any basic expects virtual methods. If you find a basic type that doesn't use virtual methods (I think res-lump will be one of these), you can add the :final option in the deftype to tell the compiler/decompiler to use non-virtual method calls.

The decompiler will now recognizes non-virtual method calls, but only if the types match up. If they don't, it leaves it as method-of-type.

Example:

((method-of-type connection get-process) obj)

is now

(get-process obj)

but only when the decompiler know the type of obj is exactly connection.

This also recognizes (new 'stack 'blah ...). Previously they were not recognized in the case where the constructor was actually called.
Example:

((method-of-type vector4s-3 new)
         (the-as symbol (new 'stack-no-clear 'vector4s-3))
         vector4s-3
         )

is now

(new 'stack 'vector4s-3)

[coveralls]

Pull Request Test Coverage Report for Build 891500464

• 60 of 88 (68.18%) changed or added relevant lines in 9 files are covered.
• 3 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.01%) to 69.773%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
decompiler/IR2/AtomicOpTypeAnalysis.cpp	2	3	66.67%
decompiler/IR2/Form.h	0	1	0.0%
common/type_system/deftype.cpp	3	10	30.0%
decompiler/IR2/FormExpressionAnalysis.cpp	29	48	60.42%

Files with Coverage Reduction	New Missed Lines	%
decompiler/IR2/Form.cpp	3	75.07%

Totals
Change from base Build 891277849:	-0.01%
Covered Lines:	35982
Relevant Lines:	51570

---

💛 - Coveralls