Xaphier Report post Posted April 22, 2011 (edited) Because of the problems with the old quickspell file format, were a corrupted file could crash the client etc, now a new format. Also at load time, more checking is done. The new format: 8 bit unsigned int with the number of spells. The number is checked at load time. For every spell only the spell id is saved as an 8 bit unsigned int. Also checked at load time so only valid ids are used. All other data is generated at load time. Feel free to post comments & suggestion. Cheers Edit: No longer using the new format Edited April 22, 2011 by Xaphier Share this post Link to post Share on other sites
bluap Report post Posted April 22, 2011 Might have been useful to request comments before you committed the code. Could we not have fixed the crash elsewhere rather than change the format? Now, its going to be a PITA switching between old and new versions until a new client is released. Could we not support the old format for a while? If the new format was written to a new file name then, if no new file exists, we can read the old file format but always write the new. Share this post Link to post Share on other sites