Stop saving "game.hardcoded_counters"

• Auswertung: cazfi
• Verantwortlicher: cazfi
• Typ: Fehler
• Status: Geschlossen
• Komponente: Server
• Meilenstein: S3_2 d3f
• Priorität: 5 - Mittel
• Schweregrad: 5 - Mittel
• Lösung: Gefixt
• Datei: 3

Details Antworten

Counters are no longer hardcoded, so we should not save "game.hardcoded_counters" any more and especially not as TRUE!

The "unused entry" warning should be silenced from dev savegames (in "< 3019300" block of compat_load_dev() )

Ticket-Verlauf (3/17 Historien)

2022-09-08 01:21 Aktualisiert von: cazfi

• New Ticket "Stop saving "game.hardcoded_counters"" created

2022-09-24 21:01 Aktualisiert von: lachu

• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10379) is attached

2022-09-24 21:02 Aktualisiert von: lachu

Kommentar Antworten

@cazfi

0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch(1KB)

I hope this address problem you're asking to solve

2022-09-24 21:08 Aktualisiert von: cazfi

Kommentar Antworten

- Remove also the loading part.
- The compat change should be one to silence "unused entry" -warning (try to run with --warnings) when the loading part is removed from the savegame3.c: likely just "(void) secfile_entry_lookup(...)"

2022-09-25 01:17 Aktualisiert von: lachu

Kommentar Antworten

Reply To cazfi

- Remove also the loading part.
- The compat change should be one to silence "unused entry" -warning (try to run with --warnings) when the loading part is removed from the savegame3.c: likely just "(void) secfile_entry_lookup(...)"

Hi. I am nowhere. I cannot found if there is a still loading part. I search for hardcoded counters in savegame3.c.

2022-09-25 01:20 Aktualisiert von: cazfi

Kommentar Antworten

Reply To lachu

I am nowhere. I cannot found if there is a still loading part. I search for hardcoded counters in savegame3.c.

Right, There wasn't. That actually was how I encountered the problem - I got the "unused entry" warning as the entry is never loaded. So; we need to stop saving the field to new savegames (your patch already did that) and to silence the warning with secfile_entry_lookup() while loading those savegames where it was saved to.

2022-09-25 01:41 Aktualisiert von: lachu

• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10387) is attached
• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10386) is attached
• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10383) is attached
• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10385) is attached
• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10384) is attached

2022-09-25 01:41 Aktualisiert von: lachu

• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10387) is deleted

2022-09-25 01:41 Aktualisiert von: lachu

• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10386) is deleted

2022-09-25 01:41 Aktualisiert von: lachu

• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10385) is deleted

2022-09-25 01:41 Aktualisiert von: lachu

• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10384) is deleted

2022-09-25 01:42 Aktualisiert von: lachu

Kommentar Antworten

Reply To cazfi

Reply To lachu

I am nowhere. I cannot found if there is a still loading part. I search for hardcoded counters in savegame3.c.

Right, There wasn't. That actually was how I encountered the problem - I got the "unused entry" warning as the entry is never loaded. So; we need to stop saving the field to new savegames (your patch already did that) and to silence the warning with secfile_entry_lookup() while loading those savegames where it was saved to.

0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch(1KB)

Change from secfile_load_bool into secfile_load_entry

This should work as expected.

2022-09-25 03:28 Aktualisiert von: cazfi

Kommentar Antworten

Looks good, but does not apply to master HEAD.

2022-09-25 18:48 Aktualisiert von: lachu

• File 0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch (File ID: 10392) is attached

2022-09-25 18:48 Aktualisiert von: lachu

Kommentar Antworten

Reply To cazfi

Looks good, but does not apply to master HEAD.

0001-OSDN-45603-S-awomir-Lach-slawek-lach.art.pl.patch(1KB)

It should apply

2022-09-25 20:01 Aktualisiert von: cazfi

• Verantwortlicher Update from (Keine) to cazfi
• Lösung Update from Keine to Accepted

2022-09-27 21:05 Aktualisiert von: cazfi

• Status Update from Offen to Geschlossen
• Lösung Update from Accepted to Gefixt