[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[patch #5825] Variable constness
From: |
Ben Pfaff |
Subject: |
[patch #5825] Variable constness |
Date: |
Wed, 28 Mar 2007 05:29:49 +0000 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.1) Gecko/20061205 Iceweasel/2.0.0.1 (Debian-2.0.0.1+dfsg-1) |
Follow-up Comment #1, patch #5825 (project pspp):
Some of this is fine. There are plenty of places where const could be used
that it isn't, and I don't object at all to adding it in those situations.
But I absolutely hate the idea of duplicating entire source files and
compiling them twice to have a "const" qualifier in one and not in the other.
It's ugly to look at, it wastes compile time, it wastes memory at runtime,
and it even potentially wastes time at runtime due to expanded cache
footprint. In return, I can't think of any corresponding benefit. Perhaps
we drop a cast or two, or avoid one or two warnings? Not at all worth it
IMO.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?5825>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/