Re: EuCOM : Attn Matt : String Return Value
Matt Lewis wrote:
>
> Bernie Ryan wrote:
> >
> > Matt Lewis wrote:
> > >
> > > Yeah, all I'm concerned about is the actual data, not the null terminator.
> > >
> > >
> > > Yeah, there should probably be two nulls appended (that's what you're
> > > asking, right?). I'm not sure why I didn't do that. It's never been
> > > a problem, though.
> > >
> >
> > Matt:
> >
> > This may cause intermitent exception errors.
> > Remember that your peek_bstr() and peek_unicode() depend on two NULLS.
>
> I agree that this should be fixed. However, I can confirm that it's not the
>
> culprit in this particular case.
>
> Matt
Perhaps could you, at the same time; fix the problem Bernie had recently pointed
out by removing any namespace references to _parts_ of a library (w32support.e,
as its name says, is not a standalone file).
Because namespaces are not inherited - if we had named packages and the ability
to namespace a package, the issue wouldn't arise at all -, namespacing part of a
library is a lock on its structure, which is bad coding unless the coder also is
the maintainer-for-life of said library. Otherwise, something has to give: the
code which uses the library - which is what just happened - or the library
itself, whose structure becomes constrained by locks put on it by programs that
use named parts of it, without the library maintainer being necessarily aware or
informed of it. Internal structures should be black boxes really.
CChris
|
Not Categorized, Please Help
|
|
