[geany/geany] Symbol windows fails in presence of 'extern "C" {' (#2793)

List overview All Threads

newer

older

[geany/geany] change gtk theme for...

[geany/geany] Plugin manager...

zipotron

21 Apr 2021 21 Apr '21

7:45 p.m.

Hello, I found a bug. In C files, in presence of the directive 'extern "C" {' Symbol window stop to show symbols, no function no variables after that, just show that is declared before. I noticed that if comment this line start working again. I would like to try to fix it escaping '{' in case of be part of 'extern "C" {' if the community consider it a proper way to fix.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793

Attachments:

attachment.htm (text/html — 1.3 KB)

Show replies by date

elextr

21 Apr 21 Apr

9:22 p.m.

`extern "C" {...}` is C++ not C, so it should not be in a C file. It is correctly parsed in C++ files.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-824170646

zipotron

22 Apr 22 Apr

3:39 p.m.

Yes, you are fully right. But I am working in my job for a project fully writen in C that, for some platform we use a C compiler but other platforms are using C++ compiler, and for keep the portability we use 'extern "C" {'. I still thinking that escape that '{' in this case for the C parser will not break any functionality and will make the life of some programmers easyer... What is your oppinion? @elextr

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-824713268

elextr

4:25 p.m.

So always edit your code as C++?

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-824738975

elextr

4:31 p.m.

What version of Geany are you using? It works for me with current git.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-824742766

zipotron

6:03 p.m.

I am using the last release, 1.37.1 Win32. I attach two captures that describes better the isue ![externC](https://user-images.githubusercontent.com/6671137/115714671-0fbb3300-a380-11...) ![noExternC](https://user-images.githubusercontent.com/6671137/115714696-1649aa80-a380-11...)

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-824799837

zipotron

9:01 p.m.

@elextr I just compiled from the git repo in Win32, I got the same issue as you can see in the captures

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-824945105

elextr

23 Apr 23 Apr

4:38 a.m.

Please post a _short_ piece of C code that fails.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-825239807

zipotron

11:25 a.m.

@elextr This is a small example, commenting and uncommenting the 'extern "C" {' you can see how appears and disappears symbols: `#ifdef LLECOM_WIFI_DEBUG #include <stdio.h> #define LLECOM_WIFI_DEBUG_TRACE printf("[DEBUG] ");printf #else #define LLECOM_WIFI_DEBUG_TRACE(...) ((void) 0) #endif

#ifdef __cplusplus extern "C" { #endif

void LLECOM_WIFI_init(void) {

WIFI_RTL8722DM_initialize_f(); }

void LLECOM_WIFI_IMPL_enableSoftAP_action(MICROEJ_ASYNC_WORKER_job_t* job) {

}

#ifdef __cplusplus } #endif

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-825406965

elextr

1:06 p.m.

Pasted your above into new document, saved to `/tmp/untitled.c`

Functions "LLECOM_WIFI_init" and "LLECOM_WIFI_IMPL_enableSoftAP_action" both show in symbols for me

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-825458967

elextr

1:06 p.m.

Ahh, can't help with windows bugs, somebody else???

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-825459259

Enrico Tröger

12 Sep 12 Sep

7:23 p.m.

Not sure why it didn't work with 1.37 on Windows but on Linux.

But now it works with current GIT master: ![geany_windows_c_symbols_issue2793](https://user-images.githubusercontent.com/617017/132990246-791a768b-24a8-400...)

@zipotron if you are curious, you can try: https://download.geany.org/snapshots/geany-1.38beta1_setup.exe https://download.geany.org/snapshots/geany-plugins-1.38beta1_setup.exe

But please note that these are unofficial builds from GIT master (with switch to GTK3 and 64bit).

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#issuecomment-917640679

Enrico Tröger

7:23 p.m.

Closed #2793.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2793#event-5289217435

1021

Age (days ago)

1165

Last active (days ago)

github-comments@lists.geany.org

12 comments

3 participants

tags (0)

participants (3)

elextr
Enrico Tröger
zipotron