[llvm-commits] MachOObjectFile fix functions

[Danil Malyshev]

Hello,

Committed as 145180, thank you!


Regards,
Danil

________________________________
From: Owen Anderson [mailto:resistor at mac.com]
Sent: Saturday, November 26, 2011 12:59 AM
To: Danil Malyshev
Cc: 'llvm-commits at cs.uiuc.edu'
Subject: Re: [llvm-commits] MachOObjectFile fix functions

Danil,

I'm good with this version.  Go ahead and apply.

--Owen

On Nov 24, 2011, at 12:05 PM, Danil Malyshev wrote:


ping

________________________________
From: Danil Malyshev
Sent: Saturday, November 12, 2011 3:00 AM
To: 'Owen Anderson'
Cc: 'llvm-commits at cs.uiuc.edu<mailto:llvm-commits at cs.uiuc.edu>'
Subject: RE: [llvm-commits] MachOObjectFile fix functions

Hello Owen

> I see your reasoning here, but it scares me a little bit to have methods with the same name that do different things.
> I think we should either rename them to be obviously different, or pick a consistent meaning for all of them.

I have renamed symbol.getOffset() as getFileOffset().
Please find attached the changed patch.

Regards,
Danil


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20111127/03e86e1a/attachment.html>