| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Codecov Report
@@ Coverage Diff @@
## master #996 +/- ##
=======================================
Coverage 86.71% 86.71%
=======================================
Files 1 1
Lines 301 301
=======================================
Hits 261 261
Misses 40 40
Continue to review full report at Codecov.
|
Sorry, something went wrong.
|
Great, thanks :) Could you also regenerate interop38.cs with this? |
Sorry, something went wrong.
output from geninterop.py updated to visit function pointers, support running on Windows 10, and 1 minor typo
|
@filmor I'm not experienced at git, so I hope I did this correctly. Happy to learn, so if I made mistakes, please let me know! |
Sorry, something went wrong.
|
The only non-ideal thing is that you changed the line-endings, the rest is fine :) I'll see whether I can easily fix this and then we'll merge, thank you very much for your contribution. It would be good if you could add yourself to the AUTHORS.md file as well. |
Sorry, something went wrong.
|
I saw that line-ending issue when Github showed me the diff. I definitely felt bad about it, but as a long-time Windows/Microsoft developer (first started on AppleSoft Basic and Microsoft's "The AppleSoft Compiler (TASC)) around 1982, at least I have an excuse. If you want me to try changing some Visual Code setting and doing it again, happy to do so! |
Sorry, something went wrong.
|
Merged manually, the whitespace modifications are apparently too much for Github to recognise this. Thanks a lot for the analysis and fix. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
What does this implement/fix? Explain your changes.
Added conditional support to run geninterop.py on Windows 10. Added support for function pointers (c_ast.FuncDecl) to fix interop38.cs for Python3.8.
Does this close any currently open issues?
close issue #995
Any other comments?
...