On 28 Jul 2008, at 8:44 PM, Adam R. Maxwell wrote: > > On Monday, July 28, 2008, at 07:56AM, "Christiaan Hofman" <[EMAIL PROTECTED] > > wrote: >> >> On 28 Jul 2008, at 3:59 PM, Adam R. Maxwell wrote: >> >>> >>> On Jul 28, 2008, at 2:13 AM, Christiaan Hofman wrote: >>> >>>> Yes, I see the warning now. Should be harmless. >>>> >>>> I inserted the newlines for saving to avoid long lines. The base64 >>>> parser in the Omni frameworks ignores garnage characters like >>>> newlines >>>> and spaces. >>> >>> [...] >>> >>>> The heuristic is as follows: anytime we have a >>>> newline >>>> * in a string, that's reason to suspect a runaway. >>> >>> [...] >>> >>>> So it's the newline in the long string in combination with the >>>> "=" at >>>> the end that triggers the warning. >>>> >>>> Anyway, it's harmless. >>> >>> In general, that warning is not harmless, so users shouldn't get in >>> the habit of ignoring it. Unfortunately, it will likely be quite >>> common now since = often appears at the end of a base64 encoded >>> string. >> >> >> Perhaps we could remove the "=" padding at the end, and re-attach >> them >> if necessary? > > That sounds worse than the warning, and a potential nightmare for > compatibility. If you implemented this, then saved a file with a > nightly build, could you then read it with 1.3.18? If not, I'd > guess that a few users would be fairly upset. Not that it matters > to me personally...I'm using openssl for base64 in my own version. > > Why add newlines in the first place? A single line is easier to > ignore in text editors that allow long lines, and updates to the > alias will only cause a single-line diff for people who use version > control systems. Inserting newlines seems like a solution in search > of a problem. >
Some programs reading bibtex have problems with long lines, as we've encountered before. We certainly should accept added newlines, because other bibtex or plain text editors may insert them. Christiaan ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Bibdesk-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/bibdesk-users
