1534530 - Remove some leftover references to RDF in comments of c-c

Status: RESOLVED FIXED

(MailNews Core :: Backend, enhancement)

Description

[:aceman]

There are still some comments in code and other text files in c-c which mention RDF even though RDF was already removed from the place or is no longer relevant.

Comment 1

[:aceman]

Attachment: 1534530-nc.patch

Remove unused nc: and rdf: namespace declarations in Thunderbird's XUL files.

Try run:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=27e4d4eb3fee5bc0811ebb04fa9a405a8930e74a

Comment 2

[:aceman]

Attachment: 1534530-ldap.patch

LDAP comments.

Comment 3

[Jorg K (CEST = GMT+2)]

Comment on attachment 9050461 [details] [diff] [review]
1534530-ldap.patch

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

::: ldap/xpcom/TODO.txt
@@ -99,5 @@
>    needs to change: assume all attributes are binary, use some
>    heuristic to figure out if they're a string.  I wonder how
>    ldapsearch does this.
>  
>  * grep for XXXs and fix the issues 

Another trailing space to be killed, or do more move into view then?

@@ -102,5 @@
>  
>  * grep for XXXs and fix the issues 
>  
> -rdf datasource 
> ---------------

Hmm, you removed the heading without removing the points below it? Are they still an issue. Do they relate to RDF?

Comment 4

[:aceman]

Yes, there are many trailing spaces in that file.
The points below the heading do not sound like particularly related to RDF so I kept them.
They just aren't implemented (or aren't to be implemented) in RDF.

Comment 5

[Jorg K (CEST = GMT+2)]

Comment on attachment 9050461 [details] [diff] [review]
1534530-ldap.patch

OK then.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/60230bc80b94
remove mensions of RDF in ldap. r=jorgk

Comment 7

[Jorg K (CEST = GMT+2)]

Attachment: 1534530-nc.patch - rebased

Comment 8

[Jorg K (CEST = GMT+2)]

Comment on attachment 9051142 [details] [diff] [review]
1534530-nc.patch - rebased

OK, thanks.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/36718c82bc31
remove unused nc: and rdf: namespace in Thunderbird's XUL files. r=jorgk