Re: Fwd: [PEAR-DEV] [PEPr] +1 for Networking::Net_IDNA

Hi Stefan, Stefan Neufeind wrote:

Hans, did you take into account the discussion we started a short while ago on pear-dev? I'd be happy if you could find the time to browse through the few postings regarding this today and express your thoughts.

Yes, I have had a look at the threads. You see, I'm not particularly traditional when it comes to weighing these packages. You may be right that certain functionality should be included, but that has little to do with my vote. Basically what I see is that: - Without having tried it, the code looks great. - Matthias seems to be a very reasonable person & sounds like he is going to make whatever changes are proven to be needed. - I tend do disagree with your desire to constantly abstract classes into uber-classes (I think you had also suggested rolling Net_Geo and Net_GeoIP into one class). Net_GeoIP is certainly an implementation class & if Matthias says this is also then that makes sense to me. - I think that php5 interfaces can solve the aforementioned problem of API, anyway. And they can be added after the fact (when there's more than 1 implementation, for example). - As for the bit about copyright, I didn't understand why that contributed to a -1 either way. - As for PHP4 support being a requirement, of course I disagree with that :) You can always create a backport for PHP4 if you desire. Anyway, I don't claim to know much about this topic; kinda intriguing, though. If the concern about letting this in is that it precludes a competing implementation, then PEAR needs to fix that issue; that IMO is no reason for a -1 to a clearly very well thought-out package. My .02, since you asked, Hans