Wireshark-dev: [Wireshark-dev] RE : column_info.h rev 23058 introduce a bug forplugins?

From: "Antoine Gardiol - FiveCo" <antoine.gardiol@xxxxxxxxx>
Date: Wed, 10 Oct 2007 11:50:56 +0200
Hello Sake,

Thanks for your answer.

For a plugin, it's a little bit a pitty to have to recompile it for each new
version of the software but ... that's life.
A good work around could be to have a function returning column number based
on ASCII column title. What do you thing about that ?

Thanks again for your help.
Best regards.
Antoine

-----Message d'origine-----
De : wireshark-dev-bounces@xxxxxxxxxxxxx
[mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] De la part de Sake Blok
Envoyé : mercredi, 10. octobre 2007 11:29
À : Developer support list for Wireshark
Objet : Re: [Wireshark-dev] column_info.h rev 23058 introduce a bug
forplugins?


Hi Antoine,

Sorry for breaking your plugin-distribution, but it has been my
understanding that all plugins need to be compiled against the sources of
the version you want to use them with. So every Wireshark version has it's
own plugin-binaries. If sometimes plugins can be used with multiple
Wireshark-versions, it's luck, because the current API does not guarantee
it.

The reason for putting the new column-options in the middle of the enums is
that they logically fit there. I would prefer to keep them there.

Cheers,


Sake
----- Original Message ----- 
From: Antoine Gardiol - FiveCo 
To: wireshark-dev@xxxxxxxxxxxxx 
Sent: Wednesday, October 10, 2007 11:07 AM
Subject: [Wireshark-dev] column_info.h rev 23058 introduce a bug for
plugins?


Hi,

I'm writting a plugin for a specific protocol of my company and I found that
a recent change to the file column_info.h (rev 23058) introduce an issue
with the check_col function when the plugin is used with a previous version
of wireshark (ie 0.99.6).

The change had added two new columns in the middle of the enum and so
COL_PROTOCOL and COL_INFO have been shifted two times.

So if the plugin is compiled with the latest release of the repository, the
check_col function will return false in all previous version of wireshark.

I think that the two new columns in the enum should be added at the end of
the list and not in the middle of it ? 

Thanks for your help.
Antoine



_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-dev