Re: Code is never executed
On 6/4/10 5:09 PM, Felix Knecht wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I *do* look at the reports, but not as frenquently as you do. Probably because I consider that we have many other serious issues to fix in the server, and I wrongly assume that they are more important than the one you are pointing out. I didn't want to blame anyone. I just do what I'm able to ;-). Both things are needed: Development and code review. As I started with this I continue and I really appreciate that you guys are going ahead with the development (and are sometimes interrupted with some 'odd' questions). We need both! Go ahead and I try to read the reports :-) Absolutely :) Just one question Is it ok to do this via dev list or shall I add jira entries which seems to me an overkill? For such Q&R, the ML is plain ok. Fill a JIRA if you feel like the question is a bit to complicated, or if you think there is a serious bug that will need love, or if you don't have time to fix it and you don't want to see the problem vanishing under a pile of mails. -- Regards, Cordialement, Emmanuel Lécharny www.nextury.com
Re: Code is never executed
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 > I *do* look at the reports, but not as frenquently as you do. Probably > because I consider that we have many other serious issues to fix in the > server, and I wrongly assume that they are more important than the one > you are pointing out. I didn't want to blame anyone. I just do what I'm able to ;-). Both things are needed: Development and code review. As I started with this I continue and I really appreciate that you guys are going ahead with the development (and are sometimes interrupted with some 'odd' questions). We need both! Go ahead and I try to read the reports :-) Just one question Is it ok to do this via dev list or shall I add jira entries which seems to me an overkill? -BEGIN PGP SIGNATURE- Version: GnuPG v2.0.15 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAkwJFzMACgkQ2lZVCB08qHFRWgCg1UxGgMAuLzw1iSnsvhGEHGHN 45cAoLXQQLPe6QQxi0ekkUWy7ZmTczOB =onsP -END PGP SIGNATURE-
Re: Code is never executed
On 6/4/10 4:40 PM, Felix Knecht wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/04/10 16:32, Emmanuel Lecharny wrote: On 6/4/10 4:14 PM, Felix Knecht wrote: This will always match for the else clause - we now the attributeType is null (line 1469). Probably dead code? http://people.apache.org/~felixk/shared-docs/xref/org/apache/directory/shared/ldap/entry/DefaultEntryAttribute.html#1475 Dohhh ! Good catch again :) Well, is it dead code now? I guess so. I will soon think that code reviews should be mandatory :) The tools are already doing this for us. I just seem to be the one ATM ( ... as I have still not much knowledge of the code ...) looking at the reports which are generated. I *do* look at the reports, but not as frenquently as you do. Probably because I consider that we have many other serious issues to fix in the server, and I wrongly assume that they are more important than the one you are pointing out. The key is : there is nothing such as frivolous errors. An error is an error, and has to be fixed. So keep going, and trust me, those reports are really important, or all of us. -- Regards, Cordialement, Emmanuel Lécharny www.nextury.com
Re: Code is never executed
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 06/04/10 16:32, Emmanuel Lecharny wrote: > On 6/4/10 4:14 PM, Felix Knecht wrote: >> This will always match for the else clause - we now the attributeType is >> null (line 1469). Probably dead code? >> >> http://people.apache.org/~felixk/shared-docs/xref/org/apache/directory/shared/ldap/entry/DefaultEntryAttribute.html#1475 >> >> > > Dohhh ! Good catch again :) Well, is it dead code now? > > I will soon think that code reviews should be mandatory :) The tools are already doing this for us. I just seem to be the one ATM ( ... as I have still not much knowledge of the code ...) looking at the reports which are generated. -BEGIN PGP SIGNATURE- Version: GnuPG v2.0.15 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAkwJEEcACgkQ2lZVCB08qHF11gCgpw0dWygqevU3sos8m5kqDR1j l5cAn2o553g8lgUE4KXoVjj130LK+n23 =NSVr -END PGP SIGNATURE-
Re: Code is never executed
On 6/4/10 4:14 PM, Felix Knecht wrote: This will always match for the else clause - we now the attributeType is null (line 1469). Probably dead code? http://people.apache.org/~felixk/shared-docs/xref/org/apache/directory/shared/ldap/entry/DefaultEntryAttribute.html#1475 Dohhh ! Good catch again :) I will soon think that code reviews should be mandatory :) -- Regards, Cordialement, Emmanuel Lécharny www.nextury.com
Code is never executed
This will always match for the else clause - we now the attributeType is null (line 1469). Probably dead code? http://people.apache.org/~felixk/shared-docs/xref/org/apache/directory/shared/ldap/entry/DefaultEntryAttribute.html#1475