cloudwu / skynet

A lightweight online game framework
MIT License
13.28k stars 4.19k forks source link

confused method name [to_little_endian] #36

Closed lparam closed 11 years ago

lparam commented 11 years ago

lua-mongo.c 里的to_little_endian方法写错了. return u.b[0] | u.b[1] << 8 | u.b[2] << 16 | u.b[3] << 24; ==> return u.b[3] | u.b[2] << 8 | u.b[1] << 16 | u.b[0] << 24;

cloudwu commented 11 years ago

你确定写错了?还是函数名没起好?

这个 driver 是可以工作的呀, 如果这个反了, 就不能正常工作了.

cloudwu commented 11 years ago

mongo 协议规定是用 little endian 的

lparam commented 11 years ago

我并没有用这个driver,只是浏览代码. to_little_endian这个方法难道不是big endian to little endian的意思么?

lparam commented 11 years ago

看了MongoDB Wire Protocol,确实只是函数名没起好.