|
bug while wrapping const char arrays without an explicitly defined size: msg#00165programming.swig
Oren Miller writes: > Using the latest CVS version: > > If I have a char array defined like this: > > const char BeginString_FIX44[8] = "FIX.4.4"; > > The wrapper for this is generated correctly like so: > > static PyObject *_wrap_BeginString_FIX44_get() { > PyObject *pyobj; > > pyobj = SWIG_FromCharArray(FIX::BeginString_FIX44, 8); > return pyobj; > } > > However, if it is defined like so: > > const char BeginString_FIX44[] = "FIX.4.4"; > > This gets generated: > > static PyObject *_wrap_BeginString_FIX44_get() { > PyObject *pyobj; > > pyobj = SWIG_FromCharArray(FIX::BeginString_FIX44, ); > return pyobj; > } > > As you can see the size is missing which of course results in > compilation errors. It would seem to me in this case Swig should count > up the characters in the string and place the value as the second > parameter to FromCharArray. I'd be happy to try to create a patch but > am not familiar with the source and would need someone to point me to > the module where this generation occurs. Thanks. > Where is SWIG_FromCharArray() coming from? I don't remember that function being part of the SWIG API (I sure didn't write it anyways). SWIG developers: Is there some reason the normal PyObject API functions are not being used here??? -- Dave _______________________________________________ Swig maillist - Swig@xxxxxxxxxxxxxxx http://mailman.cs.uchicago.edu/mailman/listinfo/swig |
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| Previous by Date: | bug while wrapping const char arrays without an explicitly defined size: 00165, Oren Miller |
|---|---|
| Next by Date: | Re: mh, it's acting up.: 00165, Martin F Krafft |
| Previous by Thread: | bug while wrapping const char arrays without an explicitly defined sizei: 00165, Oren Miller |
| Next by Thread: | Re: bug while wrapping const char arrays without an explicitly defined size: 00165, Marcelo Matus |
| Indexes: | [Date] [Thread] [Top] [All Lists] |
| News | FAQ | advertise |