1257110 - Update comm-central for PLDHashTable changes in bug 1253085

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Nicholas Nethercote [inactive]]

Bug 1253085 changed PLDHashTableOps, by removing the |PLDHashTable*| argument to the hashKey and matchEntry entries.

Here's a corresponding patch for comm-central.

Comment 1

[Nicholas Nethercote [inactive]]

Attachment: Update comm-central to handle removal of the |PLDHashTable*| argument from PLDHash{HashKey,MatchEntry}

This compiles but I haven't done any testing. It's a pretty simple patch.

Comment 2

[Kent James (:rkent)]

Thanks njn, I'll look at this once the dependent patch has landed.

Comment 3

[Kent James (:rkent)]

Comment on attachment 8731115 [details] [diff] [review]
Update comm-central to handle removal of the |PLDHashTable*| argument from PLDHash{HashKey,MatchEntry}

Review of attachment 8731115 [details] [diff] [review]:
-----------------------------------------------------------------

I can't really check that this compile because c-c is broken for other reasons, but it looks good to me so rs=me.

Comment 4

[Kent James (:rkent)]

I won't land this, but just set checkin-needed so that those who are working on getting the tree green can land it when appropriate.

Comment 5

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/143168d38573cc75f3b03813c1de5b7c3376b231
Bug 1257110 - Update comm-central to handle removal of the |PLDHashTable*| argument from PLDHash{HashKey,MatchEntry}. r=rkent. a=bustage fix CLOSED TREE DONTBUILD