Jump to content

Recommended Posts

Posted

I've always wondered why do we got two settings to do almost the same thing, can anyone explain it? maybe I'm missing something :P

//Max number of characters per account. Note that changing this setting alone is not enough if the client is not hexed to support more characters as well.
#define MAX_CHARS 9

// How many Characters are allowed per Account ? (0 = disabled)
chars_per_account: 0

Posted

9 (MAX_CHARS) is the number of slots that will be avaiable.

chars_per_account is the number of characters that you can create within MAX_CHARS slots.

Although I found that you can use any value on chars_per_account, even surpass MAX_CHARS, which is not a good practice.

Fixed that in r16937.

Posted

As far as I remember, the client can not handle arbitary number of character slots. At least old client couldn't, I do not know how the new clients behave. So I believe MAX_CHARS is there because of the client limitation. The chars_per_account is the settings users should use to limit the number of characters per account. Because modifying (especially increasing) MAX_CHARS may result in unintended behavior. But correct me if I'm wrong.

Posted

I would change the console output from:

   if (char_per_account == 0) {
       ShowStatus("Characters per Account: 'Unlimited'.\n");
   } else {
       ShowStatus("Characters per Account: '%d'.\n", char_per_account);
   }

to this:

   if( char_per_account == 0 ){
       char_per_account = MAX_CHARS;
   }

   ShowStatus( "Characters per Account: '%d'.\n", char_per_account );

Since the word "unlimited" could be missleading for some users. ;)

  • Upvote 3
Posted

I would change the console output from:

if (char_per_account == 0) {
	ShowStatus("Characters per Account: 'Unlimited'.\n");
} else {
	ShowStatus("Characters per Account: '%d'.\n", char_per_account);
}

to this:

if( char_per_account == 0 ){
	char_per_account = MAX_CHARS;
}

ShowStatus( "Characters per Account: '%d'.\n", char_per_account );

Since the word "unlimited" could be missleading for some users. ;)

Indeed, thanks for that. Done in r16939.

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...