PLUGININFO question
- Thread starter vefatica
- Start date
- May
- 13,149
- 180
Isn't that cute!
- May
- 13,149
- 180
On Sat, 03 Oct 2009 12:22:32 -0500, rconn <> wrote:
|---Quote---
|> Does it matter if PLUGININFO::pszFunctions ends with a comma?
|---End Quote---
|Leading or trailing commas are ignored.
|
|But why would you want to?
I build the list dynamically. Without having to treat the first or last
function name specially, I can do it with a simple loop.
WCHAR *p = pluginfo.pszFunctions;
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L"%s,", item.Name);
--
- Vince
|---Quote---
|> Does it matter if PLUGININFO::pszFunctions ends with a comma?
|---End Quote---
|Leading or trailing commas are ignored.
|
|But why would you want to?
I build the list dynamically. Without having to treat the first or last
function name specially, I can do it with a simple loop.
WCHAR *p = pluginfo.pszFunctions;
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L"%s,", item.Name);
--
- Vince
- May
- 13,149
- 180
On Mon, 05 Oct 2009 10:59:34 -0500, samintz <> wrote:
|You could do this instead:
| WCHAR *p = pluginfo.pszFunctions;
| for ( INT i=0; i<NUMBER_OF_ITEMS-1; i++ )
| p += Sprintf(p, L"%s,", item.Name);
| p += Sprintf(p, L"%s", item.Name);
|
Or even,
WCHAR *p = pluginfo.pszFunctions;
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L"%s,", item.Name);
*(p-1) == 0;
But not having to bother saves 16 bytes of code (in either case).
--
- Vince
|You could do this instead:
| WCHAR *p = pluginfo.pszFunctions;
| for ( INT i=0; i<NUMBER_OF_ITEMS-1; i++ )
| p += Sprintf(p, L"%s,", item.Name);
| p += Sprintf(p, L"%s", item.Name);
|
Or even,
WCHAR *p = pluginfo.pszFunctions;
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L"%s,", item.Name);
*(p-1) == 0;
But not having to bother saves 16 bytes of code (in either case).
--
- Vince
Although it is more code than nothing, this hack might be what you want as well
WCHAR *p = pluginfo.pszFunctions;
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L"%s%c", item.Name, i==NUMBER_OF_ITEMS-1 ? '\0', ',');
Or, perhaps more gawdawful, assuming TRUE/FALSE are 1/0:
WCHAR *p = pluginfo.pszFunctions;
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L"%s%c", item.Name, ","[i>=NUMBER_OF_ITEMS-1]);
On Mon, Oct 5, 2009 at 10:08 AM, vefatica <> wrote:
--
Jim Cook
2009 Saturdays: 4/4, 6/6, 8/8, 10/10, 12/12 and 5/9, 9/5, 7/11, 11/7.
Next year they're Sunday.
WCHAR *p = pluginfo.pszFunctions;
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L"%s%c", item.Name, i==NUMBER_OF_ITEMS-1 ? '\0', ',');
Or, perhaps more gawdawful, assuming TRUE/FALSE are 1/0:
WCHAR *p = pluginfo.pszFunctions;
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L"%s%c", item.Name, ","[i>=NUMBER_OF_ITEMS-1]);
On Mon, Oct 5, 2009 at 10:08 AM, vefatica <> wrote:
--
Jim Cook
2009 Saturdays: 4/4, 6/6, 8/8, 10/10, 12/12 and 5/9, 9/5, 7/11, 11/7.
Next year they're Sunday.
samintz
Scott Mintz
- May
- 1,590
- 27
Here's an even simpler approach.
Create your string by appending ",name" each time and when finished
reference ptr[1] instead of ptr[0] as the start of the string.
How does your code work? pszFunctions is a pointer not an array. Does
Sprintf() allocate memory? If not, then you're writing over memory
somewhere.
You need to have an area of memory that gtes filled in by your routine.
e.g.
WCHAR names[500]; //room for 500 characters worth of names (roughly
62 fcns x 8 chars)
WCHAR *p = &names[0];
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L",%s", item.Name);
pluginfo.pszFunctions = &names[1];
-Scott
Jim Cook <> wrote on 10/05/2009 01:53:15 PM:
Create your string by appending ",name" each time and when finished
reference ptr[1] instead of ptr[0] as the start of the string.
How does your code work? pszFunctions is a pointer not an array. Does
Sprintf() allocate memory? If not, then you're writing over memory
somewhere.
You need to have an area of memory that gtes filled in by your routine.
e.g.
WCHAR names[500]; //room for 500 characters worth of names (roughly
62 fcns x 8 chars)
WCHAR *p = &names[0];
for ( INT i=0; i<NUMBER_OF_ITEMS; i++ )
p += Sprintf(p, L",%s", item.Name);
pluginfo.pszFunctions = &names[1];
-Scott
Jim Cook <> wrote on 10/05/2009 01:53:15 PM:
- May
- 13,149
- 180
On Mon, 05 Oct 2009 13:18:40 -0500, samintz <> wrote:
|Here's an even simpler approach.
|
|Create your string by appending ",name" each time and when finished
|reference ptr[1] instead of ptr[0] as the start of the string.
I like that.
|How does your code work? pszFunctions is a pointer not an array. Does
|Sprintf() allocate memory? If not, then you're writing over memory
|somewhere.
I allocate memory for pszFunctions (free it in ShutdownPlugin()). I didn't post
that line.
--
- Vince
|Here's an even simpler approach.
|
|Create your string by appending ",name" each time and when finished
|reference ptr[1] instead of ptr[0] as the start of the string.
I like that.
|How does your code work? pszFunctions is a pointer not an array. Does
|Sprintf() allocate memory? If not, then you're writing over memory
|somewhere.
I allocate memory for pszFunctions (free it in ShutdownPlugin()). I didn't post
that line.
--
- Vince