Make sure TXT records are in same order as setup on mdns announce #1684

Merged
merged 2 commits into from Feb 7, 2013

Conversation

Projects
None yet
2 participants
Member

elupus commented Oct 26, 2012

The txtvers record in TXT records is often required to be the first record. So allow this in our API.

elupus added some commits Oct 26, 2012

@elupus elupus zeroconf: maintain txt record order when publishing service 25629fc
@elupus elupus zeroconf/osx: maintain txt field order on publish
OSX create the txt records using a dictionary, so to maintain
order, we use a custom hash function that put all elements
in the same hash bucket.
f78ada6
Owner

Memphiz commented Oct 26, 2012

Looks good and straight forward.

Owner

Memphiz commented Feb 7, 2013

@elupus - if you want that in - push that button...

@elupus elupus added a commit that referenced this pull request Feb 7, 2013

@elupus elupus Merge pull request #1684 from elupus/mdns
Make sure TXT records are in same order as setup on mdns announce
c2a21de

@elupus elupus merged commit c2a21de into xbmc:master Feb 7, 2013

Member

elupus commented Feb 7, 2013

Suppose it won't hurt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment