633315 - Make "namespace mozilla" usage consistent across DOMSVGxxxList.cpp files

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Daniel Holbert [:dholbert]]

Our usage of "namespace mozilla {" vs "using namespace mozilla;" is inconsistent between our various DOMSVG*List.cpp files.

I believe the former structure is preferred here.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: fix v1

Here's a patch to make this consistent (basically making all the other files match DOMSVGLengthList.cpp & DOMSVGAnimatedLengthList.cpp).

(Requesting approval2.0 at the same time as review.  Justification: minimal change, with no anticipated functional effect -- just keyword-usage cleanup, to help future changes that affect all of these files be marginally saner.)

Comment 2

[Daniel Holbert [:dholbert]]

http://hg.mozilla.org/mozilla-central/rev/b41a6a2499d1

Comment 3

[Jonathan Watt [:jwatt]]

Yuck, really? Why is this the preferred way? I was planing on filing a bug to do the opposite.

Comment 4

[Daniel Holbert [:dholbert]]

(In reply to comment #3)
> Yuck, really?
> I was planing on filing a bug to do the opposite.

(I'm assuming you're talking about |namespace foo {| as opposed to |using namespace foo| in .cpp files)

Yes, really -- quoting the Coding Style MDC page:
> using namespace ...; is only allowed in .cpp files after all #includes.
> Prefer to wrap code in namespace ... { ... }; instead if possible.

(Note that second line in particular.)

> Why is this the preferred way?

I'm not exactly sure, to be honest, but I have a fuzzy impression that it's cleaner to have everything inside of a namespace{} scope.  #developers or m.d.platform may be able to give you a better answer. :)