• Open Menu Close Menu
  • Apple
  • Shopping Bag
  • Apple
  • Mac
  • iPad
  • iPhone
  • Watch
  • TV
  • Music
  • Support
  • Search apple.com
  • Shopping Bag

Lists

Open Menu Close Menu
  • Terms and Conditions
  • Lists hosted on this site
  • Email the Postmaster
  • Tips for posting to public mailing lists
Re: Finder Info
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Finder Info


  • Subject: Re: Finder Info
  • From: Jens Alfke <email@hidden>
  • Date: Fri, 24 Aug 2012 22:03:51 -0700

On Aug 24, 2012, at 8:59 PM, koko <email@hidden> wrote:

> Excellent … much easier than

Well, in Carbon's defense, you're making things a lot harder than they have to be:

>        if(m_FSRef) free((void*)m_FSRef);
>        m_FSRef = malloc(sizeof(FSRef));

FSRefs don't have to be allocated on the heap. They're just structs. (Ditto with the FSCatalogInfo you use later.)

>        memset(m_FSRef,0x00,sizeof(FSRef));

There's no need to zero one out before initializing it.

>        const UInt8 *cpath = (const UInt8 *)[path cStringUsingEncoding:NSUTF8StringEncoding];

-UTF8String is shorter.

So you can reduce all the quoted stuff to one line:

	OSStatus osStatus = FSPathMakeRef((const UInt8 *)[path UTF8String], &m_FSRef, false);

assuming you declare m_FSRef as an FSRef, not FSRef*.

—Jens
_______________________________________________

Cocoa-dev mailing list (email@hidden)

Please do not post admin requests or moderator comments to the list.
Contact the moderators at cocoa-dev-admins(at)lists.apple.com

Help/Unsubscribe/Update your Subscription:

This email sent to email@hidden


  • Follow-Ups:
    • Re: Finder Info
      • From: Uli Kusterer <email@hidden>
    • Re: Finder Info
      • From: Mike Abdullah <email@hidden>
References: 
 >Re: Finder Info (From: Greg Guerin <email@hidden>)
 >Re: Finder Info (From: koko <email@hidden>)

  • Prev by Date: Re: Finder Info
  • Next by Date: Re: Finder Info
  • Previous by thread: Re: Finder Info
  • Next by thread: Re: Finder Info
  • Index(es):
    • Date
    • Thread