[Webkit-unassigned] [Bug 134186] Make it clear to get m_svgExtensions using svgExtensions()
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Aug 22 23:18:51 PDT 2014
https://bugs.webkit.org/show_bug.cgi?id=134186
--- Comment #11 from Daniel Bates <dbates at webkit.org> 2014-08-22 23:18:56 PST ---
(In reply to comment #10)
> > > Source/WebCore/svg/SVGDocumentExtensions.h:76
> > > - const HashSet<SVGFontFaceElement*>& svgFontFaceElements() const { return m_svgFontFaceElements; }
> > > + const HashSet<SVGFontFaceElement*>& svgFontFaceElements() { return m_svgFontFaceElements; }
> >
> > How did you come to the decision to make this function non-const?
> Actually, that is one of my worries and the part that I'd like to hear your opinion.
> I changed it to non-const and it's according to change of 'svgExtensions()' but I think it's not good.
> Do I need to keep 'const SVGDocumentExtensions* svgExtensions()' for this part?
I don't know at the moment. I will look through the code shortly. Can you save me some time and elaborate on the callers of svgFontFaceElements()? Regardless, it doesn't seem correct to make svgFontFaceElements() non-const given that it doesn't modify state.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list