Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

More companies (up to 16) #53

Closed
DorpsGek opened this issue Feb 11, 2006 · 6 comments
Closed

More companies (up to 16) #53

DorpsGek opened this issue Feb 11, 2006 · 6 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

wgrant opened the ticket and wrote:

A fairly small patch, enabling up to 16 companies (the limit can be changed to a maximum of 64 by just modifying MAX_PLAYERS, it is just limited now because of the lack of company colours), and 16 clients in multiplayer. The maximum number of clients could be 64 now, however that would require a large number of new strings, and a lot of bandwidth.

Scrollbars are added to the company league and detailed performance rating windows, and the graph legend windows are extended.

Attachments

Reported version: trunk
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/53
@DorpsGek
Copy link
Member Author

wgrant wrote:

The patch isn't actually in a really usable state at the moment, loading and saving are likely to not work very well at the moment, however I am working on it. Expect a new version shortly!

This comment was imported from FlySpray: https://bugs.openttd.org/task/53#comment119

@DorpsGek
Copy link
Member Author

lucaspiller wrote:

The idea seems similar to a patch I made a while ago - I dunno whether you based on mine or not, but here is a copy of mine, and a link to the forums topic: http://www.tt-forums.net/viewtopic.php?t=19808

Attachments

This comment was imported from FlySpray: https://bugs.openttd.org/task/53#comment138

@DorpsGek
Copy link
Member Author

TrueBrain wrote:

I strongly think if we are going to scale up players or companies, it should be done in a dynamic way. Not by streching 8bit variables to 64bit or what ever. If we scale it, make it unlimited (of course, memory is the limited).

This is very good possible, it just needs rewrite of code that now uses a single variable for bit-storing. Seeing this patch, it is not much code that needs altering.

This comment was imported from FlySpray: https://bugs.openttd.org/task/53#comment433

@DorpsGek
Copy link
Member Author

Rubidium wrote:

Too bad that making it unlimited is not really an option. There are limits in the OTTD source code that limit the maximum number of companies to 15.

This comment was imported from FlySpray: https://bugs.openttd.org/task/53#comment1715

@DorpsGek
Copy link
Member Author

DorpsGek commented Aug 6, 2007

TrueBrain wrote:

Then let us at least prepare the parts of the code that can be prepared for 15+ companies. Who knows, the current limitators can change in the next few revisions because of an other change.

This comment was imported from FlySpray: https://bugs.openttd.org/task/53#comment1814

@DorpsGek
Copy link
Member Author

DorpsGek commented Jan 3, 2009

Rubidium closed the ticket.

Reason for closing: Out of date

This comment was imported from FlySpray: https://bugs.openttd.org/task/53

@DorpsGek DorpsGek closed this as completed Jan 3, 2009
@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DorpsGek