<div class="gmail_quote">On 5 May 2010 08:35, Dan Bolser <span dir="ltr"><<a href="mailto:dan.bolser@gmail.com">dan.bolser@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi,<br>
<br>
It seems that the method 'readScoreMatsFromFiles' in the Scorer class<br>
of the owl.decoyScoring package should be renamed to something like<br>
'readTypeAndCountMatsFromFiles' ... or something similar.<br>
<br></blockquote><div><br>Agree.. that's probably a much better name! And by the way the java doc was not quite right so I've committed a fix for it. I must say I wrote all this code really quickly and I'm not very happy with much of it, especially with the object model. I think introducing the new interface would make things already a lot better.<br>
<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
My question is this, should I feel free to go ahead and rename<br>
methods, with the intention that my commit will be reviewed, and,<br>
where necessary, a sensible name will be picked (i.e. the method will<br>
be re-renamed), or should I initiate a discussion about this change on<br>
the mailing list (as above)?<br>
<br></blockquote><div><br>Well there's no good reply to that, it's based on feeling and experience as you would guess. It's pretty much the same as with wikipedia editing or any other type of crowd sourcing. For this case I wouldn't initiate a discussion, I'd just do it.<br>
<br><br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
I realise that it's impossible to legislate for every kind of edit,<br>
but I'm new to Java and new to the code base, so I don't have a good<br>
feel for what constitutes an insignificant, minor, significant<br>
(annoying), or major change.<br>
<br>
I tried to submit a change* which just added some comments, and I<br>
found myself writing quite a lot in the commit message (to communicate<br>
my intentions, but also asking some questions). Should I avoid putting<br>
questions there?<br></blockquote><div><br><br>Re commit comments what I tend to do these days is to describe what the change is about in a brief way without going into the details of what exact code/specific class/methods changed but rather giving an overview like: what functionality changed, a certain bug was fixed, a refactoring and the reasons for it etc. In my opinion sometimes it is important to be verbose especially for certain bug fixes, code that break some compatibility and that kind of thing. In any case if in doubt rather be verbose than not!<br>
<br></div></div>I'd say just take the plunge and experiment! By all means if you decide to rename do use eclipse's refactoring tool. It will take care of all interdependencies while refactoring plus it will warn you if something is wrong.<br>
<br>Jose<br><br>