[Ohrrpgce] SVN: james/3838 Oops! I was using Reload.rli* constants instead of Reload.rlt* constants

[James Paige]

On Thu, Oct 07, 2010 at 01:02:23AM -0400, Mike Caron wrote:
> On 10/7/2010 1:00, Mike Caron wrote:
>> On 10/7/2010 0:56, James Paige wrote:
>>> On Wed, Oct 06, 2010 at 09:55:07PM -0700,
>>> subversion at HamsterRepublic.com wrote:
>>>> james
>>>> 2010-10-06 21:55:07 -0700 (Wed, 06 Oct 2010)
>>>> 74
>>>> Oops! I was using Reload.rli* constants instead of Reload.rlt*
>>>> constants!
>>
>> Ha ha ha! You feel for my devious trap!
>>
>>
>> ...
>>
>> Honestly, though, I probably could have come up with a better name.
>> And/or not put them in reload.bi, since they don't belong there.

Heh :) Or added comments.

>>> Could anybody explain the difference between these to me? I see that the
>>> rlt constants only have one integer type, what is the purpose of the rli
>>> constants and their small multitude of different integer types?
>>
>> rli (ReLoad Internal) are the types as stored on disk. Those constants
>> are only used during loading/saving.
>>
>> rlt (ReLoad Types) are the logical types as seen by clients (number,
>> string, etc). They correspond to the GetInteger(), etc functions.

Okay, that makes sense.

> Also, just looking at the editor's code, I noticed that you cap integers  
> to be 16-bit signed. Is this a function of the grabber not supporting  
> higher, or something else (since, RELOAD supports up to 64-bit integers)?

I did that just out of bad habit.

I should probably cap to signed 32-bit numbers, since that is the actual 
size of the ints I am working with in the client.

My main goal in this editor is to make something for editing editor 
definition files, so I might not bother with every feature that reload 
supports (like the zstring buffers for example) ... at least not right 
now.

---
James