Nicklas Larsson
Nicklas Larsson
At this very moment I'm working on a PR on grass_indent_ALL :-) , and I encountered the same "indent" warnings (but not the "cmp" warnings). ```sh grass_indent_ALL.sh Indenting *.c and...
From [WP on K&R C](https://en.wikipedia.org/wiki/C_(programming_language)#K&R_C): > Compound_ assignment operators of the form =op (such as =-) were changed to the form op= (that is, -=) to remove the semantic ambiguity...
> Then I would guess `-=` and `*=` is what the code is meant to do, not `= *`. I didn't look to the source code, though. I did look...
> `Rele` branch (made by me accidenttaly) was deleted. Thanks!
Finalised the rebase, to be able to continue with clean sheets.
Opening this for review. I left the comments temporarily for context, I will remove them before merge.
This is now ready, I'll proceed to merge soon if there are no objections.
This has been addressed for RC2 (https://github.com/OSGeo/grass/commit/04af08cd45e48f4b7016366bc29826bd40317dce), remains to see if also solved (see https://lists.osgeo.org/pipermail/grass-dev/2022-January/095519.html).
https://grass.osgeo.org/grass80/binary/mswindows/native/WinGRASS-8.0.0RC2-1-Setup.exe
> This has been addressed for RC2 ([04af08c](https://github.com/OSGeo/grass/commit/04af08cd45e48f4b7016366bc29826bd40317dce)), remains to see if also solved (see https://lists.osgeo.org/pipermail/grass-dev/2022-January/095519.html). Looking closer at the screenshot, I realise my previously linked commit and mail list...