From 1278ff899409025692d6915421f1af1c99dc837e Mon Sep 17 00:00:00 2001
From: Joshua Haberman <jhaberman@gmail.com>
Date: Tue, 11 Jul 2017 14:11:54 -0500
Subject: Responded to PR comments.

---
 tools/make_c_api.lua | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

(limited to 'tools/make_c_api.lua')

diff --git a/tools/make_c_api.lua b/tools/make_c_api.lua
index aaf5d1e..c13f37c 100644
--- a/tools/make_c_api.lua
+++ b/tools/make_c_api.lua
@@ -30,11 +30,6 @@ function strip_proto(filename)
   return string.gsub(filename, '%.proto$','')
 end
 
---[[
-  [upb.TYPE_ENUM]     = 5,
-  [upb.TYPE_MESSAGE]  = 8,
---]]
-
 local function join(...)
   return table.concat({...}, ".")
 end
@@ -138,6 +133,12 @@ local function field_layout_rank(field)
   --   5. string fields
   --   6. submessage fields
   --   7. repeated fields
+  --
+  -- This has the following nice properties:
+  --
+  --  1. padding alignment is (nearly) minimized.
+  --  2. fields that might have defaults (1-5) are segregated
+  --     from fields that are always zero-initialized (6-7).
   local rank
   if field:containing_oneof() then
     rank = 4
-- 
cgit v1.2.3