--- Andy Ross <> wrote:
> Stuart Buchanan wrote:
> > - Provide a Nasal interface to display simple text messages on the
> > screen like the ATC display. In fact, I copied the code from the
> > ATCDisplay.cxx and simply shifted it further down the screen.
> >
> > The Nasal interface to display messages can be called simply as
> follows:
> > displaySingleMessage("Message here")
>
> This is really cool, but I'm not a big fan of long function names for
> simple, global actions. How about we call this thing "print()" and
> rename the existing print() function (which dumps to the console)
> log() instead? Almost everyone would prefer this to be the default
> text output, I think.
I was originally going to replicate all the ATCDisplay functions within
Nasal, hence the long name. However, in the end I only created a single
interface.
As Curt say, people expect print() to print to the console. What about
displayMessage() ?
> Also, rather than copying code from the ATC display, would it be
> better to unify the two? Is there a really good reason not to have
> both streams in the same scroll buffer?
Yes, it would be much cleaner to unify the graphics code, but I wasn't
confident enough in C++ and OpenGL to do it properly, so wimped out and
copied the code.
Additionally, I thought things might get a bit messy:
- ATC messages are only rendered if ATC is switched on - the Nasal
messages are always
- The Nasal messages are displayed below the ATC ones to avoid
over-writing.
Not brilliant reasons I know, so if you feel that it isn't an acceptable
patch as it is, let me know and I'll work on it further.
Regards,
-Stuart
___________________________________________________________
Yahoo! Messenger - NEW crystal clear PC to PC calling worldwide with voicemail
http://uk.messenger.yahoo.com
-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems? Stop! Download the new AJAX search engine that makes
searching your log files as easy as surfing the web. DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel
