search

Braini01 / bis-file-formats

c# .net standard code base for reading ArmA/BIS file formats
MIT License
46 stars 22 forks source link

Utilities to work with parseSimpleArray/string conversion methods #7

Closed jetelain closed 3 years ago

jetelain commented 3 years ago

This PR adds methods to serialize/de-serialize data to work with parseSimpleArray/string conversion methods of game engine. This can be useful to create extensions, or parse logs.

Braini01 commented 3 years ago

Thx for further contributions. Just one remark: Do you think it makes sense to add BIS.TXT as a namespace? Dont really feel that TXT files are a particular BIS file format. I think your code would be best suited in Utils or BIS.Core. What do you think?

jetelain commented 3 years ago

I was hesitating. You are right, it does not make sense. I think it will be better in BIS.Core like config, in a "Serialization" namespace. What do you think ?

Braini01 commented 3 years ago

Yeah, I also think it is closely related to Configs so it could go there.

Braini01 commented 3 years ago

LGTM! Shall I merge or are you still working on it?

jetelain commented 3 years ago

Yes, you can merge it