On 07/21/09 11:14, "Sven-S. Porst" <ssp-li...@earthlingsoft.net> wrote:

>> Why do you need a regex?  The one for
>> stringByExtractingDOIFromString is for
>> an arbitrary string that may contain a DOI somewhere.  In Coins, you
>> already
>> know it's a DOI (because of the prefix) and have the length based on
>> the
>> split string, right?
> 
> 
> That sounds reasonable to me.

Okay, I was hoping so after looking at it in the debugger; I didn't actually
read the standard, since your comments were more entertaining :).  It's the
same heuristic we've been using for years in BibItem.
 
> I think I misread the current code and interpreted as taking the
> decimalDigitCharacterSet part of the string only rather than taking
> anything _from_ the first digit. Sorry for the confusion.

Yeah, that would break badly, so thanks for double-checking.  I think that
parser was a good addition, although I had a hard time figuring out why the
citeulike parser wasn't being used on citeulike!



------------------------------------------------------------------------------
_______________________________________________
Bibdesk-develop mailing list
Bibdesk-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bibdesk-develop

Reply via email to