bugfix in rev587

Discussion Forums discussion bugfix in rev587

This topic contains 0 voices and has 2 replies.

Viewing 3 posts - 1 through 3 (of 3 total)
Author Posts
Author Posts
May 13, 2006 at 8:25 am #61

Klaus
Member

Hi Simon,

I have found 3 bugs in your sourcecode. (Rev587)

1. at the updating from Rev 586 to 587 -> File edittbr.bmp is deleted.

that generate a compiler error (file edittbr.bmp not found)

2. in file ctagsnavigator.cpp, function parseData you call: getTables(L”", &ltypes, &utypes);

-> so the ltypes and utypes ar always for cpp-Files

now the call is in PNFPGetMethods():

getTables(scheme, &ltypes, &utypes); and the ltypes and utypes gives the parseData Function.

3. The Mask for the ctags is a int (32BIT) all TAGM_xxx corresponds a Bit in the MASK

but you ar define in tagtypes.h

#define TAGM_TABLE 1 << 35 (Bit35 in a 32Bit Variable?)

now i define the Mask as 2 32Bit Variable

4. jumpview.cc the jumpToTagImages stuct needed the new tagtypes.bmp File

Icon 1 For Filename new

Icon 2 TAG_UNKNOWN

Icon 3 TAG_FUNCTION

Icon 4 TAG_PROCEDURE

Icon 5 TAG_CLASS

Icon 6 TAG_DEFINE

Icon 7 TAG_ENUM

Icon 8 TAG_MEMBER

Icon 9 TAG_PROTOTYPE

Icon 10 TAG_STRUCTURE

Icon 11 TAG_TYPEDEF

Icon 12 TAG_VARIABLE

Icon 13 TAG_NET new

Icon 14 TAG_PORT new

Icon 15 TAG_REGISTER new

Icon 16 TAG_CONSTANT new

The new icon can you change it but no remove!!

ps:

I have included the vhdl language

i send the vhdl.c File to the ctags to:

http://sourceforge.net/tracker/?group_id=6556&atid=306556

but i becomes no answer!!!! (can you contact the autor for ctags?)

The bugfix files sen i to: sourceforge.net-pnotepad-bug

May 14, 2006 at 8:39 pm #14974

simon
Key Master

Thanks Klaus, Fixed.

May 15, 2006 at 5:05 pm #14975

Klaus
Member

I have test it!

It’s ok, you can now delete the userschemesvhdl.schemedef File.

Have you now a new idea for the replace in Files function? (see “new features for the pn editor”)

or is my last integration the first step to realize this function?

Viewing 3 posts - 1 through 3 (of 3 total)

You must be logged in to reply to this topic.