On Nov 3, 2007, at 3:44 PM, Christiaan Hofman wrote:

> I don't think it's necessary to run it on headers. I think the only
> possible use could be to catch macros including NSLocalizedString,
> but I don't think they occur in any header.

I think so too.  I'm going to be changing it in the project here  
shortly.

> As for vendorsrc, all
> localized strings for the Omni frameworks should be in the special
> tables. Other 3rd party sources could contain NSLocalizedString, but
> a quick Find does not show any. Subprojects can also contain
> NSLocalizedString, in fact BTParse does. So I think that one at least
> needs to be parsed (or can it be contained in its own Resources?

I think the frameworks should all be doing their own bundled  
NSLocalizedString lookup like the Omni apps, or else they try a lookup  
from the main bundle.  I think that's the point of Omni's redefine of  
NSLocalizedString in macros.h, anyway, but I've never handled that  
appropriately in our frameworks.

> I don't know precisely where localized strings in frameworks look). So
> I think we can safely leave out most of vendorsrc. Would it be
> possible to move BDSKErrorObject.* to ./BTParse/ and just include ./*.
> {m,h} ./BTParse.{m,h} and ./inputmanager/*.{m,h}, I think that covers
> all occurences of NSLocalizedString.

The inputmanager strings shouldn't be included in BibDesk's table  
either, since those will be looked up in the containing app's bundle  
(I think) if NSLocalizedString() is used.  That one should look up the  
bundle based on identifier.

-- 
adam

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Bibdesk-develop mailing list
Bibdesk-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bibdesk-develop

Reply via email to