neslib / Chet

C Header Translator for Delphi
BSD 2-Clause "Simplified" License
204 stars 43 forks source link

Hi. I've made some improvements to the app. I hope my changes will be helpful. #18

Closed alexeydott closed 1 year ago

alexeydott commented 1 year ago
jarroddavis68 commented 1 year ago

Hmm, this looks like all of my post processing code, without any acknowledgement whatsoever????

neslib commented 1 year ago

@alexeydott Thanks for your contributions. These look very interesting! Could you rename the unit "Cnet.Postprocessor" to "Chet.Postprocessor" before I review. I will also check with Jarrod Davis if he's OK with the inclusion of his code.

neslib commented 1 year ago

@jarroddavis68 Looks like @alexeydott added comments with copyright attributions to your source files and changes. Does this address your concerns? I will not merge this pull request unless both of you agree with this.

jarroddavis68 commented 1 year ago

@neslib Yes, I have spoken to him already, all good now. Thank you both.

neslib commented 1 year ago

@neslib Yes, I have spoken to him already, all good now. Thank you both.

Thank you!

alexeydott commented 1 year ago

@neslib done)