On Tue, May 18, 2010 at 4:47 PM, Adam R. Maxwell <[email protected]> wrote:

> Okay, I checked in a slightly different fix (same result), and this time I
> compiled it with the 10.5 SDK.  It compiles and runs, so that should be a
> good sign.
>
> Must have been a while since I tried the 10.5 SDK, since it choked on my
> dispatch queue experiments also :/.  Getting availability macros right can
> be tricky.
>
> On May 18, 2010, at 7:04 AM, Adam R. Maxwell wrote:
>
> Sorry, I didn't check the macro(s) for availability.  The mach_vm_*
> functions are available back to 10.4, at least in the xnu sources, so I
> figured mach_vm_round_page was introduced at the same time.
>
> On May 18, 2010, at 3:12 AM, Christiaan Hofman wrote:
>
> Be aware that I reverted these changes. They were only valid with the 10.6
> SDK, which we're not using.
>
> Christiaan
>
> On Tue, May 18, 2010 at 11:57 AM, Christiaan Hofman <[email protected]>wrote:
>
>> Adam, I am currently getting compiler errors from FileView due to your
>> recent changes. It complains about mach_vm_round_page not being defined. I
>> don't know where this is defined, I cannot find it in mach_vm.h.
>>
>> Christiaan
>>
>>
> Is it safe to access the extra two fields in the malloc_zone_t based just
on the SDK version? I think that will fail on 10.5, i.e. the deployment
target. I don't know whether it's relevant to set the free_definite_size
callback?

Christiaan
------------------------------------------------------------------------------

_______________________________________________
Bibdesk-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bibdesk-develop

Reply via email to