[Mono-dev] MonoTODO attribute usage policy.

Charlie Poole charlie at nunit.com
Thu Dec 7 12:28:12 EST 2006


Would one or more separate attributes serve better? Would it be better if
MonoTODO were called MonoNotSupported and just did one job?
 
In following the discussion, it struck me that the naming of the attribute
was an open invitation to use it in ways it wasn't intended to be used.
 
Charlie


  _____  

From: mono-devel-list-bounces at lists.ximian.com
[mailto:mono-devel-list-bounces at lists.ximian.com] On Behalf Of Eyal Alaluf
Sent: Thursday, December 07, 2006 3:20 AM
To: Miguel de Icaza; Ximian Mono List; mono-devel-list at lists.ximian.com
Cc: Philippe Cohen; Noam Lampert
Subject: Re: [Mono-dev] MonoTODO attribute usage policy.



Hi, Miguel.

 

We at Mainsoft would like to extend the MonoTODO for documentation purposes
and for the benenfit of tools (like NOMA).

For that end I propose the following change:

  Define - enum MonoTODOCategory { NotSupported, Limitation,
DocumentationNote, Extension, InternalRemark }

  Add a ctor to MonoTODO attribute with the enum as the first param.

This way it will be easy to identify the reason the MonoTODO was added to
the code and use it properly in the documentation or tool.

Any comments are welcome.

 

Eyal.

 

-----Original Message-----
From: mono-devel-list-bounces at lists.ximian.com
[mailto:mono-devel-list-bounces at lists.ximian.com] On Behalf Of Miguel de
Icaza
Sent: 14 November 2006 23:34
To: Ximian Mono List; mono-devel-list at lists.ximian.com
Subject: [Mono-dev] MonoTODO attribute usage policy.

 

Hello,

 

    MonoTODO attributes have been used in the past for two purposes: to

warn us about things that must be reviewed (this shows up on the class

status pages) and for comments that a developer would use.

 

    Since the Mono Meeting, we discussed using MonoTODOs instead as a

mechanism to provide a textual description that would assist a developer

that might use the API.  So the description should now be only to be

consumed by a user of the API (this is what we actually had documented

in the Wiki).

 

    This means that implementation notes like "We should improve X or

Y", do not belong in MonoTODO attributes.  Use "FIXME" strings on the

code, or if the issue is one worth following up with, then file a bug in

Bugzilla.

 

    I have updated the 

 

      http://www.mono-project.com/Coding_Guidelines

 

    page.

 

Miguel.

_______________________________________________

Mono-devel-list mailing list

Mono-devel-list at lists.ximian.com

http://lists.ximian.com/mailman/listinfo/mono-devel-list

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ximian.com/pipermail/mono-devel-list/attachments/20061207/b4d48eed/attachment.html 


More information about the Mono-devel-list mailing list