Hello zyx, some updates below.
On Mon, 27 Feb 2023 at 10:16, Francesco Pretto <[email protected]> wrote: > Then PdfPainter::GetContent() is fine. Done. > I will take inspiration from .net GraphicsPath.AddPath Done[1]. On Thu, 23 Feb 2023 at 14:02, Francesco Pretto <[email protected]> wrote: > On Thu, 23 Feb 2023 at 08:32, zyx <[email protected]> wrote: > > You changed it to `common`, `main`, > > `private`, `staging` and apart of `staging` > > I believe there's some more rationale for the separation I picked, I > explained everything in the SOURCE-LAYOUT.md document After some thoughts, I decided to rename "common" to "auxiliary" [2] and finished populating it with all the left files that were not prefixed "Pdf": a few actually existed in 0.9.x, but all of the ones that ended in this folder were created for 0.10 or renamed. While this endures on the categorization of the source files, it should be much more clear what ends in "main" and what should be put on "auxiliary", according to the definition in SOURCE_LAYOUT.md. There are some border line cases, but in the end the rule should be enforceable enough as I don't think much more should/will end in "auxiliary". Greetings, Francesco [1] https://github.com/podofo/podofo/blob/04206218e0bf2c6c750d55b0469d45c0fafe1650/src/podofo/main/PdfPainterPath.cpp#L79 [2] https://github.com/podofo/podofo/tree/master/src/podofo/auxiliary On Mon, 27 Feb 2023 at 10:16, Francesco Pretto <[email protected]> wrote: > > On Mon, 27 Feb 2023 at 09:42, zyx <[email protected]> wrote: > > I'd say the "String()" part of the function is kinda redundant, because > > the API shows it returns a string. Nonetheless, I've no strong opinion, > > the GetContentString() sounds good too. > > > > Then PdfPainter::GetContent() is fine. I'll go for it. > > > Thinking of > > it now, would it work to construct a path and apply it multiple times > > into the drawing context? > > PdfPainterPath is meant to be reusable of course. > > > I do not say to add a "copy from other Path" is a needed thing, it only > > felt like a good thing to me. > > I think it's a perfectly reasonable feature and easy to add. I will > take inspiration from .net GraphicsPath.AddPath[1]. > > Regards, > Francesco > > [1] > https://learn.microsoft.com/en-us/dotnet/api/system.drawing.drawing2d.graphicspath.addpath _______________________________________________ Podofo-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/podofo-users
