Xavier Wang

I have actived the travis-ci of this project. maybe you could just push another commit to see whether it works :)

it seems that your script install moonglfw failed: ```shell $ bash ./scripts/build.sh Setting up runtime configuration Detected PLATFORM=linux machine=Linux extension=so lua=5.3 Building in /home/travis/build/starwing/lua-nanovg/build/linux Ensuring build directory in /home/travis/build/starwing/lua-nanovg/build/linux Building...

你这样直接加了field的排序真的没问题么……？都没有处理pb_newfield/pb_delfield啊……先不说多分配一块内存了（这个好像都不是很重要了），这种排序，添加删除field的时候怎么维护……

这个还是要全局想想，我之前不太想做这个的一个主要原因就是内存分配。另外排序可能问题不大，目前唯一会修改的地方是load，load结束再排序就可以。还有一个问题，扩展extension是会插入到其他类型的，如果每次都排序，那么排序复杂度是$O(n^2lgn)$，这肯定是不能接受的。

那倒是可以。可以考虑比如说如果有field修改就置位一个标记，如果需要排号序的发现有标记就排个序，这样应该可以。

我看到很多地方调用了gettop，不过其实没有必要，只要允许负索引，然后实现一个lpb_relindex函数就好了，具体实现是，如果索引是正的或者小于等于LUA_REGISTRYINDEX，那么保持不变，否则，减去一个传入的当前栈上多余的元素数量即可。这个技巧是我在其他项目的源码里面学会的，能够少一次跨dll调用（并且减少某些函数长度……我如果有可能，我都是希望函数最多25行的）

还有就是，目前代码的函数名是Lua风格的——也就是说，除了函数名前缀，所有字母小写并且不用下划线。说实在的有点不太好认，但是短，而且强迫你想又短又好认的函数名（比如lpbE_pack就足够了，或者lpb_packmsg也行）

说到风格，else不换行也是（刚开始我也不太习惯，不过现在觉得不换行好像更好看一点）

> 关于调用了gettop这个，之前的做法是用pushvalue来复制一份到栈顶，所以可以传负索引或者说不传索引，默认使用-1。现在用gettop是因为想省去做这个复制。这两种做法，我确实也有点疑问它们差别有多大，最终测出来的结果是差别太小，看不出来。如果想用pushvalue的方式，我可以改一下 啊，不是这个意思，首先其实差别不大，其次pushvalue的方式还是有损耗嘛，我说的意思是这样的： ```c static int lpb_relindex(lua_State *L, int idx, int onstack) { return idx >= 0 || idx 命名这个，是按上一版本写的。我原本考虑是用pack、unpack，但现在pb.pack、pb.unpack这两函数是被占用了，所以之前有问一下是否考虑把现有的pack、unpack接口放回buffer和slice模块，然后这两个接口用pack、unpack。如果不改，那我可以按现有的风格改成lpbE_packmsg、lpbD_unpackmsg 就直接换掉好了，好像的确没啥人用。

> `static int lpb_relindex(lua_State *L, int idx, int onstack)`需要另一个参数`onstack`，要么就用现在的gettop来取，要么在对应的函数再加一个参数，或者放到`lpb_Env *e`中来维护，感觉都不算优雅。有什么其他好办法么？ 并不需要哇……你还是没理解这个方法，当你调用一个传入了idx的函数的时候，唯一会导致idx出错的情况是你动了栈（比如push了或者pop了啥），你在**这个函数**里面当然是知道自己干了啥的，所以你自己数函数里面怎么个动法，然后做一个常量传给`lpb_relindex`就行了，比如你知道在之前的操作里你往栈上推了5个值，你就传5，如果你推了3个值，你就传3，如果你pop了2个值（并且这两个值都是在idx之后的）那么你就传-2。这个函数就可以帮你算出在这个位置正确的idx值。 比如上面的例子，假设你调用`dosomething(L, -2)`，当函数运行到`lua_settable`的时候，作为`dosomething`函数的作者你当然知道，从这个函数开始到这个地点，你对栈的操作是“push了两个值”，所以你传了2，然后idx就变成了 (-2)-2 也就是 -4，正好是你在push了两个值以后，idx指向的那个位置的正确索引。