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

Many changes #1

Merged
merged 24 commits into from
Jun 9, 2019
Merged

Many changes #1

merged 24 commits into from
Jun 9, 2019

Conversation

barncastle
Copy link
Contributor

  • Added a solution file and some new projects
    • Unit Test (can remove was just for testing... funnily enough)
    • DBFileReaderLib
  • Added DBFileReaderLib so the lib supports reading
    • Supports all DB types and now DBCache.bin
    • purposely segregated as per @Marlamin's request
  • Fixes
    • Added a field accessor for DBCDRows

@Marlamin
Copy link
Contributor

Marlamin commented Jun 8, 2019

Minor note, neither DBCD or DBFileReaderLib can be compiled as Release due to missing AllowUnsafeBlocks for Release configs in project files.

I also just pushed a new version of DBDefsLib that doesn't try to open files for writing when reading.

@Marlamin
Copy link
Contributor

Marlamin commented Jun 8, 2019

Crashes when parsing soundkitname.db2 from 8.2.0.30669 and possibly more builds. Issue seems to be centered around stringTable reading. Crashes on line 59/60 of WDC3Reader.cs with : 'The given key '503511' was not present in the dictionary.'.

@barncastle
Copy link
Contributor Author

@Marlamin these should be fixed with the above two commits

@barncastle barncastle merged commit 5953d10 into wowdev:master Jun 9, 2019
Marlamin pushed a commit that referenced this pull request Aug 23, 2024
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.

2 participants