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

Add: Change profileNamespace DB to IniDB #83

Closed
wants to merge 23 commits into from
Closed

Conversation

Vdauphin
Copy link
Owner

@Vdauphin Vdauphin commented Jan 17, 2016

Giallustio#72

DOC of https://github.com/code34/inidbi2 : https://github.com/code34/inidbi2/blob/master/%40inidbi2/DOCUMENTATION.txt

  • Add : IniDB.
  • FIX : IniDB do not save object (only array, string, boolean) (side, getVariable "inHouse").
  • FIX : Array save too big for IniDB (>8k).

Take the array [city1,city2,city3] and check if the array is to big. If it is, I reduce the size of it and save separately the array in two keys ("cities1","cities2"):

  • cities1 = "[city1,city2]"

  • cities2 ="[city3]"

    The load phase is easy just read ("cities1","cities2") and append .

need to test :

Final test :

  • local
  • server

@Vdauphin Vdauphin added this to the 1.14 milestone Jan 17, 2016
@Vdauphin Vdauphin changed the title Add: Change DB for InitDB Add: Change DB for IniDB Jan 17, 2016
Vdauphin pushed a commit that referenced this pull request Jan 31, 2016
FIX: btc_patrol_active and btc_civ_veh_active negative
@Vdauphin Vdauphin changed the title Add: Change DB for IniDB Add: Change profileNamespace DB to IniDB Feb 3, 2016
@Vdauphin
Copy link
Owner Author

Vdauphin commented Feb 6, 2016

DB save took 30s to save the game. Here is the .ini output file :
btc_hm_Altis_30s.txt
The same game has been saved with #85 and took much time (4 minutes see #85 (comment))

@Vdauphin Vdauphin closed this Apr 25, 2016
@Vdauphin Vdauphin deleted the Add-initDB branch May 20, 2017 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant