[Bug 491518] Review Request: openttd - Transport system simulation game

bugzilla at redhat.com bugzilla at redhat.com
Sun May 17 11:47:20 UTC 2009


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=491518





--- Comment #12 from Hans de Goede <hdegoede at redhat.com>  2009-05-17 07:47:19 EDT ---
(In reply to comment #10)
> (In reply to comment #6)
> > > - Would be good to generate AI docs from src/ai/api/Doxyfile and put them to
> > > subpackage.
> > > 
> > 
> > This IMHO makes no sense, this is an internal API, which is not exported
> > so no need to buid or include the docs.
> 
> This is only script "external" API that is used by squirrel AI scripts. And it
> could be used by AI writers as scripts are easily added as plugins.
> 

Ah, ok then indeed it would be good to package this in a sub-package.

> > (In reply to comment #8)
> > > (In reply to comment #6)
> > > > As promised I will review this, but it looks like we first need to address
> > > > the flags issue <sigh>, assigning to me.  
> > > 
> > > I'm thinking, that we can just hide language column with flags. If such
> > > solution would be acceptable, I can come with a patch for it.  
> > 
> > Would there then still be a way for the user to find out the server language
> > from the pick a server UI ?  
> 
> Yes, when user clicks on individual server line, server language will be shown
> on server details panel. But in such, there won't be simple way to fast search
> for particular language - user will have to click on each server if he wants to
> find server where is that particular language spoken.  

Hmm, not ideal but I guess just hiding the flags column in the UI is an ok
solution then.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list