bufanliu / apparat

Automatically exported from code.google.com/p/apparat
0 stars 0 forks source link

Abstract/Concrete only works with public functions? #38

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create base class with protected [Abstract] function
2. Create inherited class with implementation of Abstract function
3. Run concrete tool on swf

What is the expected output? What do you see instead?
I expect that concrete would correctly identify the function as implemented. 
However, it still returns the "must implement abstract method" error.

What version of the product are you using? On what operating system?
apparat-1.0RC8, Windows XP SP 3

Please provide any additional information below.

Original issue reported on code.google.com by darren...@gmail.com on 19 Nov 2010 at 3:10

GoogleCodeExporter commented 9 years ago
Can you please show the code snippet you were using. It should not be limited 
to public functions, but of course, it can only work with public/protected 
methods.

Original comment by joaebert on 19 Nov 2010 at 3:12

GoogleCodeExporter commented 9 years ago

The attached code is compiled into a SWC, but I also tried with a regular SWF, 
with no luck. In the same SWC, public methods marked as abstract work correctly.

Thanks!

Original comment by darren...@gmail.com on 19 Nov 2010 at 3:19

Attachments:

GoogleCodeExporter commented 9 years ago
Is the tool saying that Processed must implement CreateInstance?

Original comment by joaebert on 19 Nov 2010 at 4:28

GoogleCodeExporter commented 9 years ago
Correct:
 [concrete] [ERROR] Class framework.message.game.Processed must implement abstract method CreateInstance

Original comment by darren...@gmail.com on 19 Nov 2010 at 5:25

GoogleCodeExporter commented 9 years ago

Original comment by joaebert on 20 Dec 2010 at 9:24

GoogleCodeExporter commented 9 years ago
Protected namespaces equality check was not okay in Concrete.

Original comment by joaebert on 1 Feb 2011 at 7:56