Re: [luxor-xul-develop] ChromeClassResourceLoader & ChromeResourceLoaderManager

[Gerald B. <ge...@va...>]

Hello David Carr,

> I noticed that these files had numerous fix and todo
> comments to allow
> proper loading of config and bootstrap entries. 
> Attached are versions
> with implementations for those methods.  The index
> file used by the
> ClassResourceLoader is just a properties file; the
> keys are ignored, and
> all values are added to the entries.

  Thanks for sending in your patch. I invite you to
fix up your property lookup. The property lookup is
supposed to be locale/culture sensitive, that is,
depending on your default locale you will get
different values back. Also I don't see a need for a
separate property index file. Can you explain why it's
needed?

  Let me know if you're intersted in improving your
patch. It shouldn't be too hard. If you have any
questions let me know.

  - Gerald