SicroAtGit / PB-CodeArchiv-Rebirth

A collection of useful codes from the PureBasic forums and other sources.
Other
72 stars 22 forks source link

Improve DisplayASMCode and add DisplayCCode #13

Closed mk-soft-65 closed 2 years ago

mk-soft-65 commented 2 years ago
SicroAtGit commented 2 years ago

Thank you for the PR.

I saw your post in the German PureBasic forum, but didn't have enough time to look at it in detail until now.

  • Check and correction of environment variable "PB_TOOL_Compiler"

I wonder if it doesn't make more sense if we wait until this PB_TOOL_Compiler update delayed bug in the PureBasic IDE is fixed. Then we could rename this tool to Display-ASM/C-Code or something like that and keep everything in one tool.

If I have enough time, I will have a look at the corresponding PureBasic IDE code and try to fix the bug.

  • PB Editor standard constants added

Thanks.

  • Output to standard editor and clipboard the asmCode without own header

We could add a checkbox gadget for this to make it optional (default is then all content, i.e. checkbox gadget is set). I think that some users want to take the whole content that is displayed in the textbox gadget.

  • Include files dependencies resolved

I prefer to keep dependencies external to avoid duplicate code and to only have to make improvements to the dependency code once in one place for each dependency.

SicroAtGit commented 2 years ago

I have now created a pull request with the PB_TOOL_Compiler fix in the PureBasic IDE repository.

SicroAtGit commented 2 years ago

I have now made the remaining changes and merged the commits into meaningful commits.

@mk-soft-65 Thanks a lot for your PR!