Important RDF Progids not defined in nsI*.h files

RESOLVED WONTFIX

Status

()

Core
RDF
P3
minor
RESOLVED WONTFIX
18 years ago
18 years ago

People

(Reporter: edburns, Assigned: Chris Waterson)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
The following progids should be defined in nsI*.h files.  I've had to manually 
define them in order to access the components they reference:

static const char *NS_RDFSERVICE_PROGID = "component://netscape/rdf/rdf-
service";
static const char *NS_CONTAINERUTILS_PROGID 
= "component://netscape/rdf/container-utils";
static const char *NS_RDFCONTAINER_PROGID 
= "component://netscape/rdf/container";

Comment 1

18 years ago
reassigning to brendan and ccing waterson
Assignee: endico → brendan
QA Contact: nobody → endico
Sounds like an RDF bug.

/be
Assignee: brendan → waterson
Progids should not be in nsI*.h files.  progIDs identify implementations, and
the ``I'' in nsI*.h stands for ``interface''.

You want an rdf-progids.h, perhaps, which lists all the progids for our RDF
components.  You don't want (and, more strongly, I don't want for you =) )
implementation labels mixed into interface headers, though this misguided
practice isn't uncommon in our tree.

(Would you have all of the widget progids in nsIWidget.h?  All the progids in
the product tacked onto the end of nsISupports.h?  Folly, I say.)
(Assignee)

Comment 4

18 years ago
Sure, I'll put a bunch of #defines into mozilla/rdf/base/public/rdf.h. Sounds
like a good idea.
Status: NEW → ASSIGNED
Product: Architecture → Browser
Target Milestone: --- → M16
Version: unspecified → other
(Assignee)

Comment 5

18 years ago
Ed, can you give me patches? thanks...
Assignee: waterson → edburns
Status: ASSIGNED → NEW
Target Milestone: M16 → ---
(Reporter)

Comment 6

18 years ago
I don't think a patch will help, since my changes were in the file 

mozilla/java/webclient/src_moz/rdf_util.h

but here's the relevent data:

rdf_util.h:static const char *NS_IRDFSERVICE_PROGID =
"component://netscape/rdf/rdf-service";
rdf_util.h:static const char *NS_ICONTAINERUTILS_PROGID =
"component://netscape/rdf/container-utils";
rdf_util.h:static const char *NS_IBOOKMARKSSERVICE_PROGID =
"component://netscape/browser/bookmarks-service";
rdf_util.h:static const char *NS_IRDFCONTAINER_PROGID =
"component://netscape/rdf/container";


I'm reassigning this to waterson.
Assignee: edburns → waterson
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → M18
(Reporter)

Comment 7

18 years ago
Chris, any progress on this?

Ed
(Assignee)

Comment 8

18 years ago
i'm not going to fix this.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.