@techee commented on this pull request.


In ctags/main/parse.c:

> @@ -2742,8 +2793,12 @@ extern void anonGenerate (vString *buffer, const char *prefix, int kind)
 
 	vStringCopyS(buffer, prefix);
 
-	unsigned int uHash = anonHash((const unsigned char *)getInputFileName());
-	sprintf(szNum,"%08x%02x%02x",uHash,lang -> anonumousIdentiferId, kind);
+/* GEANY DIFF */
+/*	unsigned int uHash = anonHash((const unsigned char *)getInputFileName());
+	sprintf(szNum,"%08x%02x%02x",uHash,lang -> anonumousIdentiferId, kind); */
+	sprintf(szNum,"%u", lang -> anonumousIdentiferId);

The problem with it is that in the symbol tree you'll see anonymous types like:

anon_enum_162324ab457c
anon_enum_8734cb98f023
anon_enum_fc820958ba67

instead of

anon_enum_1
anon_enum_2
anon_enum_3

and the latter is nicer for users.

We already detect whether a tag is anonymous in tm_tag_is_anon() and don't perform autocompletion for it so there's no problem with the same anonymous names from different files.

Even this is not perfect - anonumousIdentiferId is used for all anonymous types so we'll get things like

anon_enum_1
anon_struct_2
anon_enum_3

and it would be nicer to have

anon_enum_1
anon_struct_1
anon_enum_2

But in the new cxx parser the anonymous name isn't different for individual types and the above would be

Anonymous1
Anonymous2
Anonymous3

so at least for C and C++ this problem will disappear.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.