Opened 5 months ago

Closed 4 months ago

#16154 closed enhancement (fixed)

add wxGraphicsRenderer::GetName

Reported by: aasselin Owned by: VZ
Priority: normal Milestone: 3.1.0
Component: GUI-all Version:
Keywords: Cc:
Blocked By: Blocking:
Patch: yes

Description

this patch adds wxGraphicsRenderer::GetName for Cairo/CoreGraphics/GDI+ imeplementations

Attachments (2)

graphics-getname.patch download (3.8 KB) - added by aasselin 5 months ago.
graphics-getname-getversion.patch download (5.5 KB) - added by aasselin 5 months ago.

Download all attachments as: .zip

Change History (6)

Changed 5 months ago by aasselin

comment:1 Changed 5 months ago by aasselin

in fact I wonder if I should not include the version of the library (being the OS version for GDI+ / CoreGraphics, and Cairo version for Cairo), as the behavior of the renderer may not be the same... or should I add as well wxGraphicsRenderer::GetVersionString() ?

comment:2 Changed 5 months ago by vadz

  • Milestone set to 3.1.0
  • Status changed from new to confirmed

Thanks, just two minor comments:

  1. We've recently added wxOVERRIDE, could you please use it for the overridden methods?
  2. Is it really a good idea to use "+" in the string meant for the file names? Maybe it should return "gdiplus"? Generally speaking, if it's not fit for the human consumption anyhow, why capitalize them? I'd go with "cairo" and "cg" then (and maybe add GetDisplayName() later if we really need it).

TIA!

Changed 5 months ago by aasselin

comment:3 Changed 5 months ago by aasselin

1 and 2, OK, I had to add GetVersion as well, to let the drawing tests know about which version of the renderer they use.

comment:4 Changed 4 months ago by VZ

  • Owner set to VZ
  • Resolution set to fixed
  • Status changed from confirmed to closed

In 76380:

Add wxGraphicsRenderer::GetName() and GetVersion() methods.

Allow the code to determine which underlying technology is used for
implementing wxGraphics API. This is needed by the unit tests to account for
the known differences between platforms and may be useful in other cases.

Closes #16154.

Note: See TracTickets for help on using tickets.