Either that or explicitly make the paramater of type SolutionTask. As I said I'm not sure if the intent was to have it callable by other tasks than the solution task.I agree with your points on the cast. My general approach is to try to keep the number of parameters passed as small as possible to reduce coupling. I certainly should not have cast without checking the type. I was also concerned about not changing things too much on my first try.
Anyway, I'm guessing you would prefer me to pass the FileSet and other
properties I need from the task to the classes as individual parameters.
Is this the normal design pattern?
I don't think so. That should be fine. I used to use brace under the if as well but now that I've been using the other way I appreciate the extra vertical space.I'll set VS.NET to convert the tabs to spaces and I'll move the braces to where the coding guidelines suggest (although my religion has always been braces under the if :-) ). Was there anything else that did not fit the guidelines?
Ian
------------------------------------- TFC
-----Original Message-----
From: Ian MacLean [mailto:[EMAIL PROTECTED] Sent: Tuesday, August 12, 2003 10:50 PM
To: Tom Cabanski
Cc: [EMAIL PROTECTED]
Tom,
the diff you sent has tabs instead of spaces and incorrect brace positioning. Could you please read the coding guidelines at http://sourceforge.net/docman/display_doc.php?docid=6080&group_id=31650 and re-submit.
Another question I have is about the nantTask paramater passed in to the
Solution class's constructor. In your patch you cast this to a SolutionTask instance and then use some properties from it. This works fine now because Solution is only ever instanciated from SolutionTask passing itself in the constructor.
However in that case why is the paramater a Task instance instead of a SolutionTask insance ? Is there an intent to use the solution class from
other tasks that require solution functionality ? With these changes Solution will break if somthing other than a SolutionTask is passed to the constructor. The same goes for ConfigurationSettings.
maybe Matt can comment on the intended use of the _nanttask member.
Ian
A couple weeks ago I submitted a patch that added webmap to the solution task and fixed a couple of bugs including the issue in Resource.cs that made resource compilations failed. I just finished getting latest from the anonymous source forge and none of this has been applied. Should I be looking elsewhere?
We identified another bug in the resource compilation of the solution task that I wanted to fix.
Please advise.
-------------------------------------
TFC
-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01
_______________________________________________
nant-developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-developers
------------------------------------------------------- This SF.Net email sponsored by: Free pre-built ASP.NET sites including Data Reports, E-commerce, Portals, and Forums are available now. Download today and enter to win an XBOX or Visual Studio .NET. http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01 _______________________________________________ nant-developers mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/nant-developers
