Horde3D

Next-Generation Graphics Engine
It is currently 18.04.2024, 18:38

All times are UTC + 1 hour




Post new topic Reply to topic  [ 4 posts ] 
Author Message
 Post subject: beef: using directive
PostPosted: 20.10.2008, 18:24 
Offline

Joined: 20.10.2008, 17:32
Posts: 2
Hello Horders,

Despite the compactness and the shiny, I have a gripe:
"using namespace ..." in header files.

If there are no objections, I will task an underling with the job of removing the offending code and qualifying the headers.

R.


Top
 Profile  
Reply with quote  
PostPosted: 20.10.2008, 19:19 
Offline

Joined: 23.06.2008, 10:23
Posts: 21
Location: Sweden
I agree the "using namespace" should be removed from the headers, namespaces are there for a reason.


Top
 Profile  
Reply with quote  
PostPosted: 20.10.2008, 23:07 
Offline

Joined: 22.11.2007, 17:05
Posts: 707
Location: Boston, MA
Full agreement - using is sort of OK in cpp files, but not safe in headers.

_________________
Tristam MacDonald - [swiftcoding]


Top
 Profile  
Reply with quote  
PostPosted: 24.10.2008, 20:57 
Offline

Joined: 20.10.2008, 17:32
Posts: 2
Here's a patch to remove the using directives from the headers.

It goes on top of the SVN repository found here:
http://mm-werkstatt.informatik.uni-augs ... ic/Horde3D

Rob.


Attachments:
File comment: no using patch
Horde3D_no_using_namespace_std.zip [15.42 KiB]
Downloaded 557 times
Top
 Profile  
Reply with quote  
Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 4 posts ] 

All times are UTC + 1 hour


Who is online

Users browsing this forum: No registered users and 12 guests


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
cron
Powered by phpBB® Forum Software © phpBB Group