[webkit-dev] …Inlines.h vs …InlineMethods.h

Eric Seidel eric at webkit.org
Fri Nov 2 17:55:43 PDT 2012


My understanding was that *Inlines/*InlineMethods were more about
limiting includes in the main header.  Maybe that's just a happy side
effect. :)

I also prefer the *Inlines naming. :)

On Fri, Nov 2, 2012 at 5:48 PM, Mark Lam <mark.lam at apple.com> wrote:
> FYI, some time in the near future (maybe this weekend), I plan to do some work to break inline methods in JavaScriptCore out of header files into their own inline header files.
>
> Naming-wise, the existing JSC code has a few inline headers named …Inlines.h and more named …InlinedMethods.h.  On the WebCore side, the few that exists there are named …InlineMethods.h.  I have a preference for the …Inlines.h convention because it is shorter and concisely communicates the intent.  I plan to rename all these inline headers to be …Inlines.h for consistency.  Does anyone object to my renaming the inline headers?
>
> Also, a lot of the existing inlined methods are 1 liners.  I plan to leave these in the original header file.  Here's the convention I'm going to adopt:
>
> 1. If the inline method is a 1 liner, then leave it in the original header file.
> 2. If it takes more than 1 line, then move it into a corresponding inline header file.
>
> I will only be breaking out the inline methods (per the above convention) in the JSC sources.  Apart from renaming the few WebCore inline headers, I won't be looking into WebCore's placement of inline methods for now.
>
> If anyone has an opinion on this, please let me know asap.  Thanks.
>
> Regards,
> Mark
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo/webkit-dev


More information about the webkit-dev mailing list