[jitsi~svn:9117] Add a combined A/AAAA lookup


#1

Hey Ingo,

Just a quick one:

     /**
+ * Returns array of hosts from the A or AAAA record of the specified domain.

<snip>

+ public static InetSocketAddress[] getAorAAAARecords(String domain, int port)

If I understand correctly the method would return all A and AAAA record.
When I first saw the name however I assumed the "or" in the title and
the comment meant that I would receive either A or AAAA records. How about:

    getAandAAAARecords(String domain, int port)

Emil


#2

Hey

You're right :slight_smile:
I'll change it with one of the next commits.

Thanks for having a look,
Ingo

···

-----Original Message-----
From: Emil Ivov [mailto:emcho@jitsi.org]
Sent: Donnerstag, 17. November 2011 15:09
To: Jitsi Developers
Subject: [jitsi-dev] Fwd: [jitsi~svn:9117] Add a combined A/AAAA lookup

Hey Ingo,

Just a quick one:

> /**
> + * Returns array of hosts from the A or AAAA record of the specified
domain.

<snip>

> + public static InetSocketAddress[] getAorAAAARecords(String domain, int
port)

If I understand correctly the method would return all A and AAAA record.
When I first saw the name however I assumed the "or" in the title and
the comment meant that I would receive either A or AAAA records. How about:

    getAandAAAARecords(String domain, int port)

Emil