unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#23919: hideif.el
@ 2016-07-08 13:57 Kenneth Johansson
  2020-12-15  6:58 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 3+ messages in thread
From: Kenneth Johansson @ 2016-07-08 13:57 UTC (permalink / raw)
  To: 23919

GNU Emacs 24.5.1 (x86_64-pc-linux-gnu, GTK+ Version 3.18.9) of 
2016-04-17 on lgw01-04, modified by Debian

I got a result I did not expect where the hide-ifdef mode was hiding 
more than it should .


here is the test file
-------------------------
main()
{
#if defined(A)
         not_this_A();
#else
#if defined (B)
         not_this_B();
#else
#if defined (C)
         this();
#endif
#endif
#endif

#if defined(A)
         not_this_A();
#elif defined (B)
         not_this_B();
#elif defined (C)
         this();
#endif

}
-------------------------


now doing hide when no variables is defined shows the correct result. 
but when you define C then only the top variant shows the this() 
function call and that do not look right to me. if this is intended then 
the whole mode is more or less useless.

tried to test the emacs version in git but it miss compiled so I do not 
know if this is already fixed.








^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#23919: hideif.el
  2016-07-08 13:57 bug#23919: hideif.el Kenneth Johansson
@ 2020-12-15  6:58 ` Lars Ingebrigtsen
  2021-01-22 18:32   ` Lars Ingebrigtsen
  0 siblings, 1 reply; 3+ messages in thread
From: Lars Ingebrigtsen @ 2020-12-15  6:58 UTC (permalink / raw)
  To: Kenneth Johansson; +Cc: 23919

[-- Attachment #1: Type: text/plain, Size: 428 bytes --]

Kenneth Johansson <kenneth@southpole.se> writes:

> now doing hide when no variables is defined shows the correct
> result. but when you define C then only the top variant shows the
> this() function call and that do not look right to me. if this is
> intended then the whole mode is more or less useless.

(This bug report unfortunately got no response at the time.)

I tried reproducing the test case in Emacs 28, and I got:


[-- Attachment #2: Type: image/png, Size: 52910 bytes --]

[-- Attachment #3: Type: text/plain, Size: 192 bytes --]


Which looks correct to me.  Are you still seeing this problem in newer
Emacs versions?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#23919: hideif.el
  2020-12-15  6:58 ` Lars Ingebrigtsen
@ 2021-01-22 18:32   ` Lars Ingebrigtsen
  0 siblings, 0 replies; 3+ messages in thread
From: Lars Ingebrigtsen @ 2021-01-22 18:32 UTC (permalink / raw)
  To: Kenneth Johansson; +Cc: 23919

Lars Ingebrigtsen <larsi@gnus.org> writes:

> Which looks correct to me.  Are you still seeing this problem in newer
> Emacs versions?

More information was requested, but no response was given within a
month, so I'm closing this bug report.  If the problem still exists,
please respond to this email and we'll reopen the bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-01-22 18:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-08 13:57 bug#23919: hideif.el Kenneth Johansson
2020-12-15  6:58 ` Lars Ingebrigtsen
2021-01-22 18:32   ` Lars Ingebrigtsen

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).