Dekel Tsur <[EMAIL PROTECTED]> writes:
| > | Since this code works, it appears that fl_addtopup creates an internal
| > | copy of the string, which means that we don't need the string pool.
|
| I also tried the following code
| char buffer[200];
| for (LastFiles::const_iterator cit = lastfiles->begin();
| cit != lastfiles->end() && ii < 10; ++cit, ++ii) {
| string tmp = tostr(ii);
| tmp += ". " + MakeDisplayPath((*cit), 30);
| strcpy(buffer, tmp.c_str());
| fl_addtopup(FileMenu, buffer);
|
| and it worked. Now this MUST means that fl_addtopup keeps a local copy of
| the string (at least in XForms 0.88 on which I tested).
Yes, that should be an conclusive test.
A patch to remove the use of strpool will be accepted. Don't remove
the StrPool from the repository just yet.
Lgb
