On Sun, Apr 25, 2010 at 4:56 PM, Christiaan Hofman <[email protected]> wrote:
>
> On Apr 25, 2010, at 20:59, Patrik Jonsson wrote:
>
>> On Sun, Apr 25, 2010 at 1:31 PM, Patrik Jonsson
>> <[email protected]> wrote:
>>> In any case, it was easy to patch skim to swap the order of reading so
>>> it first reads the notes from the .skim file instead of the EAs. If
>>> anyone is interested in this, the patch is attached. The notes are
>>> still *saved* to the EAs, so if something should happen to the
>>> "fragile" files they should still be in the EAs. The meaning of the
>>> hidden preference "SKReadMissingNotesFromSkimFileOptionKey" is
>>> inverted so it controls whether the notes are read from the EAs if no
>>> skim file is found.
>>
>> The patch I just sent has a mismatched brace causing the preferences
>> option to not work as expected. This one should work.
>>
>> Sorry about that,
>>
>> /Patrik
>> <skim_read_from_file.patch>
>
>
> I'm not saying it's not possible to do it. I'm saying it's not going to 
> happen. The discussion probably did not mention the extension .skim an in any 
> way because it predated the existence of such files. The conclusion was the 
> current support for .skim files, which was a compromise. And the maximum that 
> will be offered. I'm not going to repeat that discussion again (and again and 
> again and again).

I didn't mean for the patch to be included in the "official" version,
but to be used if anyone else (like Jeremy apparently) needs it to
work like I do and want to build their own version.  That's the beauty
of open-source, after all.

Regards,

/Patrik

------------------------------------------------------------------------------
_______________________________________________
Skim-app-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/skim-app-users

Reply via email to