And a quick comment - back in the day I split OutputDevice into seperate files. People were actually very supportive of this change. You asked me why I split out the files, but bitmap functions are already split. So how is it useful? Well, we could combine all the functions into one file, because bitmap3.cxx as a filename is less than useful.
I have been consulting with Tomasz about my code changes, by the way. He has been working on the bitmap code, and I’ve been writing bitmap tests, which apparently are useful. If you don’t like the unit tests, please let me know - after all, I’ve stoped looking at the SAL as it got people’s noses out of joint. Given I’ve previously worked on the VCL and even fixed bugs, I thought that would be alright. I have some other changes I’ve suggested, I’m going to submit them to gerrit when I get back from visiting my brother in Victoria. Stephan, are you actively working on vcl bitmap code? I am happy to add you as a reviewer to my gerrit commits, would you like me to do this in future? Are there any other modules you want me to add you as a reviewer? Chris Sent from my iPad > On 6 Mar 2018, at 8:25 pm, Stephan Bergmann <sberg...@redhat.com> wrote: > >> On 17.11.2017 10:03, Stephan Bergmann wrote: >> * Don't reformat when moving an existing file >> When moving an old file that is listed in solenv/clang-format/blacklist (so >> doesn't automatically get reformatted upon commit), adapt >> solenv/clang-format/blacklist to contain the new name, instead of provoking >> a reformat upon commit. Otherwise, if the reformatted code looks >> sufficiently different from the original, it's hard to impossible to keep >> track in the git history of which file was moved where. > > Please also follow this advice when moving part of an existing file out to a > new file, as happened e.g. at > <https://cgit.freedesktop.org/libreoffice/core/commit/?id=bdd62b4c327ab894dbba00fe2e07696c1b7d9de6> > "vcl: split painting bitmap functions to bitmappaint.cxx". > _______________________________________________ > LibreOffice mailing list > LibreOffice@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/libreoffice _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice
