Re: [Arianne-devel] How about adding floats as attribs?

[Erik K. <kru...@uq...>]

SkyFlash wrote:
> We talked about that before... it may give Arianne another speed boost.

> I think adding INTs is a waste of time as most numbers can still be 
> added as floats (unless we would need really big numbers which we 
> don’t). Adding arbitrary variables is kinda a larger project, and I dont 
> feel like adding yet another library to Arianne, its too many already.

Once you add one, it's trivial to add another.  Think of how STL does
this with various _traits classes to get at the underlying types.
You don't need to hard-code things the C++ compiler already "knows"
about.

Do you mean compiled library, or header-file "library"?  My proposal
uses only .h files.  There is no external library to accomplish any
of what the tarball shows you how to do.

By what date is it necessary to implement these changes?  Just last
week, you told me not to do it.

> But how about adding the possibility to add floats as attribs to the 
> Objects and Actions?

As long as you wrap things up so only the wrapper needs to be changed.
Hopefully "grep -r '\<float\>'" will always return extremely sparse
results :)

> Please comment on it.... we can always later when we got time on hands 
> make it arbitrary.

--comment--
I have some time I could use for this, seeing as it's just implementation
code (as opposed to knowing how all the pieces are utilised).  I'd rather
see you working on [global] function rather than implementation details
that have narrow, directed, "smells like optimization" scope.
-----------

What's your position on interior vs. exterior properties on objects?
I.e., is there a use to having access to "all objects with x,y position"
or "all objects with an intelligence attribute"?

---------
tons of warnings in compiling newarianne, but only one fatal error.
Out of curiousity, why do you not include 'Kyra/' for kyra include files?
(The fatal error was that I have guiExtended/progress.h instead
  of gui/progress.h in my .../include/Kyra/.  No big deal.)
> 
> SkyFlash
>