* bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__
@ 2011-07-13 15:03 Dan Nicolaescu
2011-07-14 18:27 ` Lars Magne Ingebrigtsen
` (3 more replies)
0 siblings, 4 replies; 12+ messages in thread
From: Dan Nicolaescu @ 2011-07-13 15:03 UTC (permalink / raw
To: 9066
The source now assumes standard C, there are a few places that test __STDC__,
those tests are not needed anymore.
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__
2011-07-13 15:03 bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__ Dan Nicolaescu
@ 2011-07-14 18:27 ` Lars Magne Ingebrigtsen
2011-07-14 18:57 ` Glenn Morris
2011-07-17 0:36 ` Lars Magne Ingebrigtsen
` (2 subsequent siblings)
3 siblings, 1 reply; 12+ messages in thread
From: Lars Magne Ingebrigtsen @ 2011-07-14 18:27 UTC (permalink / raw
To: Dan Nicolaescu; +Cc: 9066
Dan Nicolaescu <dann@gnu.org> writes:
> The source now assumes standard C, there are a few places that test __STDC__,
> those tests are not needed anymore.
This was marked "notabug" by... somebody... so I'm closing the report.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog http://lars.ingebrigtsen.no/
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__
2011-07-14 18:27 ` Lars Magne Ingebrigtsen
@ 2011-07-14 18:57 ` Glenn Morris
2011-07-14 19:06 ` Lars Magne Ingebrigtsen
0 siblings, 1 reply; 12+ messages in thread
From: Glenn Morris @ 2011-07-14 18:57 UTC (permalink / raw
To: Lars Magne Ingebrigtsen; +Cc: Dan Nicolaescu, 9066
Lars Magne Ingebrigtsen wrote:
> Dan Nicolaescu <dann@gnu.org> writes:
>
>> The source now assumes standard C, there are a few places that test __STDC__,
>> those tests are not needed anymore.
>
> This was marked "notabug" by... somebody... so I'm closing the report.
Closing is not appropriate. I marked it as notabug, because it isn't an
actual "bug", IMO. But it is a TODO item, and this tracker is used for
keeping track of those as well. So it should stay open, and actually get
fixed after the feature freeze ends.
I mean personally I wouldn't file this kind of trivia as a tracker item,
but if people want to...
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__
2011-07-14 18:57 ` Glenn Morris
@ 2011-07-14 19:06 ` Lars Magne Ingebrigtsen
2011-07-14 19:27 ` Glenn Morris
0 siblings, 1 reply; 12+ messages in thread
From: Lars Magne Ingebrigtsen @ 2011-07-14 19:06 UTC (permalink / raw
To: Glenn Morris; +Cc: Dan Nicolaescu, 9066
Glenn Morris <rgm@gnu.org> writes:
> Closing is not appropriate. I marked it as notabug, because it isn't an
> actual "bug", IMO. But it is a TODO item, and this tracker is used for
> keeping track of those as well. So it should stay open, and actually get
> fixed after the feature freeze ends.
I'm thinking "notabug" is what other bug tracking systems say is
"invalid". That is, the report talks of something that is the way it
should be, a la, "doctor, when I poke a finger in my eye it hurts".
I'll reopen the bug.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog http://lars.ingebrigtsen.no/
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: cleanup: remove conditional preprocessor code that depends on __STDC__
2011-07-13 15:03 bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__ Dan Nicolaescu
2011-07-14 18:27 ` Lars Magne Ingebrigtsen
@ 2011-07-17 0:36 ` Lars Magne Ingebrigtsen
2011-07-18 5:56 ` bug#9066: " Dan Nicolaescu
2011-07-18 5:56 ` Dan Nicolaescu
2011-07-17 0:36 ` bug#9066: " Lars Magne Ingebrigtsen
2013-02-14 8:10 ` Glenn Morris
3 siblings, 2 replies; 12+ messages in thread
From: Lars Magne Ingebrigtsen @ 2011-07-17 0:36 UTC (permalink / raw
To: Dan Nicolaescu; +Cc: 9066, emacs-devel
Dan Nicolaescu <dann@gnu.org> writes:
> The source now assumes standard C, there are a few places that test __STDC__,
> those tests are not needed anymore.
I've had a peek at these, and they seem pretty trivial.
But is the time for cleanups like this now, or should they wait until
after the next release?
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog http://lars.ingebrigtsen.no/
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__
2011-07-17 0:36 ` Lars Magne Ingebrigtsen
@ 2011-07-18 5:56 ` Dan Nicolaescu
2011-07-18 5:56 ` Dan Nicolaescu
1 sibling, 0 replies; 12+ messages in thread
From: Dan Nicolaescu @ 2011-07-18 5:56 UTC (permalink / raw
To: Lars Magne Ingebrigtsen; +Cc: 9066, emacs-devel
Lars Magne Ingebrigtsen <larsi@gnus.org> writes:
> Dan Nicolaescu <dann@gnu.org> writes:
>
>> The source now assumes standard C, there are a few places that test __STDC__,
>> those tests are not needed anymore.
>
> I've had a peek at these, and they seem pretty trivial.
>
> But is the time for cleanups like this now, or should they wait until
> after the next release?
IMO, just wait until the next release, they are not worth the risk.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: cleanup: remove conditional preprocessor code that depends on __STDC__
2011-07-17 0:36 ` Lars Magne Ingebrigtsen
2011-07-18 5:56 ` bug#9066: " Dan Nicolaescu
@ 2011-07-18 5:56 ` Dan Nicolaescu
1 sibling, 0 replies; 12+ messages in thread
From: Dan Nicolaescu @ 2011-07-18 5:56 UTC (permalink / raw
To: Lars Magne Ingebrigtsen; +Cc: 9066, emacs-devel
Lars Magne Ingebrigtsen <larsi@gnus.org> writes:
> Dan Nicolaescu <dann@gnu.org> writes:
>
>> The source now assumes standard C, there are a few places that test __STDC__,
>> those tests are not needed anymore.
>
> I've had a peek at these, and they seem pretty trivial.
>
> But is the time for cleanups like this now, or should they wait until
> after the next release?
IMO, just wait until the next release, they are not worth the risk.
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__
2011-07-13 15:03 bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__ Dan Nicolaescu
2011-07-14 18:27 ` Lars Magne Ingebrigtsen
2011-07-17 0:36 ` Lars Magne Ingebrigtsen
@ 2011-07-17 0:36 ` Lars Magne Ingebrigtsen
2013-02-14 8:10 ` Glenn Morris
3 siblings, 0 replies; 12+ messages in thread
From: Lars Magne Ingebrigtsen @ 2011-07-17 0:36 UTC (permalink / raw
To: Dan Nicolaescu; +Cc: 9066, emacs-devel
Dan Nicolaescu <dann@gnu.org> writes:
> The source now assumes standard C, there are a few places that test __STDC__,
> those tests are not needed anymore.
I've had a peek at these, and they seem pretty trivial.
But is the time for cleanups like this now, or should they wait until
after the next release?
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog http://lars.ingebrigtsen.no/
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__
2011-07-13 15:03 bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__ Dan Nicolaescu
` (2 preceding siblings ...)
2011-07-17 0:36 ` bug#9066: " Lars Magne Ingebrigtsen
@ 2013-02-14 8:10 ` Glenn Morris
2013-02-16 14:01 ` Eli Zaretskii
3 siblings, 1 reply; 12+ messages in thread
From: Glenn Morris @ 2013-02-14 8:10 UTC (permalink / raw
To: 9066
Dan Nicolaescu wrote:
> The source now assumes standard C, there are a few places that test
> __STDC__, those tests are not needed anymore.
The only ones left are in
src/w32term.c
nt/inc/ms-w32.h
OK to remove, or still needed?
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__
2013-02-14 8:10 ` Glenn Morris
@ 2013-02-16 14:01 ` Eli Zaretskii
0 siblings, 0 replies; 12+ messages in thread
From: Eli Zaretskii @ 2013-02-16 14:01 UTC (permalink / raw
To: Glenn Morris; +Cc: 9066
> From: Glenn Morris <rgm@gnu.org>
> Date: Thu, 14 Feb 2013 03:10:44 -0500
>
> Dan Nicolaescu wrote:
>
> > The source now assumes standard C, there are a few places that test
> > __STDC__, those tests are not needed anymore.
>
> The only ones left are in
>
> src/w32term.c
> nt/inc/ms-w32.h
>
> OK to remove, or still needed?
I removed the first one, but the 2nd one has to stay. I made it clear
it is only used for the MS compiler, though; GCC doesn't need it.
Trunk revision 111805.
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-02-16 14:01 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-13 15:03 bug#9066: cleanup: remove conditional preprocessor code that depends on __STDC__ Dan Nicolaescu
2011-07-14 18:27 ` Lars Magne Ingebrigtsen
2011-07-14 18:57 ` Glenn Morris
2011-07-14 19:06 ` Lars Magne Ingebrigtsen
2011-07-14 19:27 ` Glenn Morris
2011-07-14 19:29 ` Lars Magne Ingebrigtsen
2011-07-17 0:36 ` Lars Magne Ingebrigtsen
2011-07-18 5:56 ` bug#9066: " Dan Nicolaescu
2011-07-18 5:56 ` Dan Nicolaescu
2011-07-17 0:36 ` bug#9066: " Lars Magne Ingebrigtsen
2013-02-14 8:10 ` Glenn Morris
2013-02-16 14:01 ` Eli Zaretskii
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.