MantisBT - Doomseeker
View Issue Details
0003797Doomseeker[All Projects] Cleanuppublic2020-06-05 09:042020-06-07 19:40
WubTheCaptain 
 
nonetrivialhave not tried
feedbackopen 
1.3.1 
 
0003797: huffman.cpp is mostly C, despite the file extension, but with C++ namespace, using C's stdlib.h
$ file src/plugins/zandronum/huffman/huffman.cpp
src/plugins/zandronum/huffman/huffman.cpp: C source, ASCII text, with CRLF line terminators

This is from upstream, do I care to fix rename this to huffman.c?
Compare:
$ file src/plugins/zandronum/huffman/bitwriter.cpp
src/plugins/zandronum/huffman/bitwriter.cpp: C++ source, ASCII text

Interestingly, huffman.cpp has C++ namespace (using namespace skulltag;) but a C header (#include <stdlib.h>) instead of C++ equivalent. So I guess I could change it to #include <cstdlib>.
I'm trying to make the whole source tree more browsable and friendly to searching with grep. Hence, this question / issue while cleaning up.
No tags attached.
Issue History
2020-06-05 09:04WubTheCaptainNew Issue
2020-06-07 19:08Pol MNote Added: 0021375
2020-06-07 19:08Pol MNote Edited: 0021375bug_revision_view_page.php?bugnote_id=21375#r13152
2020-06-07 19:09Pol MAssigned To => Pol M
2020-06-07 19:09Pol MStatusnew => feedback
2020-06-07 19:40Pol MAssigned ToPol M =>

Notes
(0021375)
Pol M   
2020-06-07 19:08   
From what I can tell, this file comes from here and we should try to keep them in sync in order to easily pull changes from upstream. You can propose the change to the zandronum project and send them a pull request, and then we can merge it in.