Bug 318802 (MONO75883) - [PATCH] mcs fails to accept partial abstract classes when 'abstract' isn't specified in all partial declarations
Summary: [PATCH] mcs fails to accept partial abstract classes when 'abstract' isn't sp...
Status: RESOLVED FIXED
Alias: MONO75883
Product: Mono: Compilers
Classification: Mono
Component: C# (show other bugs)
Version: 1.1
Hardware: Other Linux
: P3 - Medium : Enhancement
Target Milestone: ---
Assignee: Atsushi Enomoto
QA Contact: Mono Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-23 16:37 UTC by Itamar Rogel
Modified: 2007-09-15 21:24 UTC (History)
1 user (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments
A testcase for mcs (try to compile with "mcs pa.cs") (472 bytes, text/plain)
2005-08-23 16:38 UTC, Thomas Wiest
Details
proposed fix + testcases (6.62 KB, patch)
2005-09-01 16:09 UTC, Thomas Wiest
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Wiest 2007-09-15 19:28:13 UTC


---- Reported by chastamar@yahoo.com 2005-08-23 09:37:02 MST ----

Description of Problem:

mcs does not deal properly with the 'abstract' modifier for partial
classes. Version 2.0 of the C# standard states:
"If one or more partial declarations of a class include an abstract
modifier, the class is considered abstract (ยง10.1.1.1). Otherwise, the
class is considered non-abstract." (see
http://www.ferca.com/c_2.0_specification.html, 23.1.2)

Steps to reproduce the problem:
1. Try to compile the attached test... (below)
 
Actual Results:
Failure to compile (Error: CS0262)

Expected Results:
The program would compile and run.

How often does this happen? 
All the time.

Additional Information:



---- Additional Comments From chastamar@yahoo.com 2005-08-23 09:38:51 MST ----

Created an attachment (id=168404)
A testcase for mcs (try to compile with "mcs pa.cs")




---- Additional Comments From atsushi@ximian.com 2005-09-01 09:09:16 MST ----

Created an attachment (id=168405)
proposed fix + testcases




---- Additional Comments From rharinath@novell.com 2005-09-01 09:43:06 MST ----

Man, nice patch :-)  Looks ok to commit.  I'm assuming this has passed
a 'make compiler-tests'.

Make sure you update known-issues-gmcs.  (You'll find the list of
problem cases in gmcs.log.  Otherwise, you can run the buildbot --
it'll be somewhere in the test-profile log)




---- Additional Comments From atsushi@ximian.com 2005-09-01 10:47:17 MST ----

heh, thanks (I didn't add known issues as well as ChangeLogs just
because it could be midair collision). Now the patch is in svn (r49269).

Imported an attachment (id=168404)
Imported an attachment (id=168405)