Re: [PR] Remove unneeded blank lines from cxx files [openoffice]

2024-12-31 Thread via GitHub
Pilot-Pirx merged PR #238: URL: https://github.com/apache/openoffice/pull/238 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@openoffice

Re: [PR] Remove unneeded blank lines from cxx files [openoffice]

2024-12-31 Thread via GitHub
Pilot-Pirx commented on code in PR #238: URL: https://github.com/apache/openoffice/pull/238#discussion_r1900206798 ## main/automation/source/server/recorder.cxx: ## Review Comment: Often German Umlauts were replaced: ä -> ae ö -> oe ü -> ue ß -> ss I also saw

Re: [PR] Remove unneeded blank lines from cxx files [openoffice]

2024-12-31 Thread via GitHub
Pilot-Pirx commented on code in PR #238: URL: https://github.com/apache/openoffice/pull/238#discussion_r1900206798 ## main/automation/source/server/recorder.cxx: ## Review Comment: Often German Umlauts were replaced: ä -> ae ö -> oe ü -> ue ß -> ss I also saw

Re: [PR] Remove unneeded blank lines from cxx files [openoffice]

2024-12-31 Thread via GitHub
Pilot-Pirx commented on code in PR #238: URL: https://github.com/apache/openoffice/pull/238#discussion_r1900206798 ## main/automation/source/server/recorder.cxx: ## Review Comment: Often German Umlauts were replaced: ä -> ae ö -> oe ü -> ue ß -> ss I also saw

Re: [PR] Remove unneeded blank lines from cxx files [openoffice]

2024-12-31 Thread via GitHub
jbampton commented on code in PR #238: URL: https://github.com/apache/openoffice/pull/238#discussion_r1900205357 ## main/automation/source/server/recorder.cxx: ## Review Comment: Some of these German words have found their way into our codespell ignored words list in #191

Re: [PR] Remove unneeded blank lines from cxx files [openoffice]

2024-12-31 Thread via GitHub
Pilot-Pirx commented on code in PR #238: URL: https://github.com/apache/openoffice/pull/238#discussion_r1900203619 ## main/automation/source/server/recorder.cxx: ## Review Comment: Yes, still old German comments from the time where no Unicode was used... ;-) -- This is

Re: [PR] Remove unneeded blank lines from cxx files [openoffice]

2024-12-31 Thread via GitHub
jbampton commented on code in PR #238: URL: https://github.com/apache/openoffice/pull/238#discussion_r1900199682 ## main/automation/source/server/recorder.cxx: ## @@ -586,7 +586,7 @@ IMPL_LINK( MacroRecorder, EventListener, VclSimpleEvent*, pEvent )

[PR] Remove unneeded blank lines from cxx files [openoffice]

2024-12-31 Thread via GitHub
jbampton opened a new pull request, #238: URL: https://github.com/apache/openoffice/pull/238 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-