Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Numpsy opened a pull request on ironfede/openmcdf
Fix paramter order for ArgumentException
I thought this had already been done :-(
Numpsy created a branch on Numpsy/openmcdf
3.0/param_names - Microsoft Compound File .net component - pure C# - NET Standard 2.0
Numpsy opened a pull request on ironfede/openmcdf
Make PropertyIdentifierAndOffset internal
I think it should be treated as an implementation detail of the (already internal) PropertySet/PropertySetStream
Numpsy created a branch on Numpsy/openmcdf
3.0/private_offsets - Microsoft Compound File .net component - pure C# - NET Standard 2.0
Numpsy opened a pull request on ironfede/openmcdf
Make IProperty and related internal
As discussed elsewhere - IProperty is basically an implementation detail of the property factory machinery so I think it can all be made internal
Numpsy pushed 25 commits to 6.0/internal_property Numpsy/openmcdf
- Rework the handling of FMTID0 in OlePropertiesContainer a311f64
- Add large write perf methods a5ff835
- Update MSTest to v3.6.3 d2227d0
- Add coverlet to tests da2287d
- Improve Header coverage bb87cb3
- Improve EntryInfo coverage 7a3aa64
- Add Seek test 51f8db0
- Add read/write span tests 449df1e
- Add docs from 2.4 48b6034
- Improve DirectoryEntry coverage f40e612
- Remove redundant code 2e9be2b
- Add tests for creating/opening files 929656f
- Improve Storage coverage 3d2e728
- Fix flushing with consolidation 73cb2ce
- Add tests for Storage metadata 5c9f220
- Test creating entries with invalid names be4fd11
- Exclude validation test methods from code coverage c61c16c
- Add code coverage improvements a4a545d
- Test consolidation with a file efc294c
- Improve DirectoryTree test coverage e811d99
- and 5 more ...
Numpsy pushed 24 commits to master Numpsy/openmcdf
- Rework the handling of FMTID0 in OlePropertiesContainer a311f64
- Add large write perf methods a5ff835
- Update MSTest to v3.6.3 d2227d0
- Add coverlet to tests da2287d
- Improve Header coverage bb87cb3
- Improve EntryInfo coverage 7a3aa64
- Add Seek test 51f8db0
- Add read/write span tests 449df1e
- Add docs from 2.4 48b6034
- Improve DirectoryEntry coverage f40e612
- Remove redundant code 2e9be2b
- Add tests for creating/opening files 929656f
- Improve Storage coverage 3d2e728
- Fix flushing with consolidation 73cb2ce
- Add tests for Storage metadata 5c9f220
- Test creating entries with invalid names be4fd11
- Exclude validation test methods from code coverage c61c16c
- Add code coverage improvements a4a545d
- Test consolidation with a file efc294c
- Improve DirectoryTree test coverage e811d99
- and 4 more ...
Numpsy created a comment on a pull request on ironfede/openmcdf
I'll close it - can do a seprate pass over the Ole bits for span usage later
Numpsy closed a draft pull request on ironfede/openmcdf
[POC] Add some Span based write functions to CFStream
An experiment for #57 to see how much can be added by just rearranging the internal functions, without having to duplicate blocks of code for the array/span cases. Currently just does the sugges...
Numpsy opened a draft pull request on ironfede/openmcdf
Simplify null terminator handling in DictionaryProperty.Write
I believe this isn't needed after #124, where the read side was changes to remove all the null terminators, and the write side of various other string properties was changed to always write null te...
Numpsy created a branch on Numpsy/openmcdf
3.0/simpler_dict_write - Microsoft Compound File .net component - pure C# - NET Standard 2.0
Numpsy created a review comment on a pull request on ironfede/openmcdf
A thought when looking at this - is there actually a need to keep the stream as a member?
Numpsy created a review comment on a pull request on ironfede/openmcdf
There was a previous suggestion to add a constructor that allows users to specify the fmtid to use rather than inferring it such that values the library doesn't know about can be used, but I don't ...
Numpsy opened a pull request on ironfede/openmcdf
Rework the handling of FMTID0 in OlePropertiesContainer
As #142 but against v3.0 and with an extension to one of the unit tests to test that it retains the fmtid for a file with an 'app specific' (i.e not a known value) value,
Numpsy pushed 1 commit to 3.0/fmtid2 Numpsy/openmcdf
- Rework the handling of FMTID0 in OlePropertiesContainer 21fc528
Numpsy created a branch on Numpsy/openmcdf
3.0/fmtid2 - Microsoft Compound File .net component - pure C# - NET Standard 2.0
Numpsy created a comment on an issue on ironfede/openmcdf
To follow up on my previous question in https://github.com/ironfede/openmcdf/pull/216#discussion_r1845258198 - I wonder if there is actually a need for IProperty itself to be public - instances...
Numpsy created a branch on Numpsy/openmcdf
6.0/internal_property - Microsoft Compound File .net component - pure C# - NET Standard 2.0
Numpsy pushed 7 commits to master Numpsy/openmcdf
- Add TryOpenStream/Storage 17626c3
- Reduce Storage DirectoryEntry access f4bf712
- Add some definitions of known property set names 6316bb0
- Simplify the reading of DictionaryProperty e3cd19d
- Mark the .NET 8 versions of the libraries as Trim and AOT compatible da09df8
- Set ContinuousIntegrationBuild to true in CI builds d5ac1d0
- Make sure PropertyNames is always populated in AddUserDefinedProperty() a23eecf
Numpsy created a review comment on a pull request on ironfede/openmcdf
Basically I'm not clear why DictionaryProperty would be public when all the other implementations of IProperty (i.e the TypedPropertyValue classes) appear to be internal. Usage of those classes ...
Numpsy opened a pull request on ironfede/openmcdf
Mark the .NET 8 versions of the libraries as Trim and AOT compatible
Should enable the trimming and AOT code analyzers so we get warnings if any incompatible features creep it