[Ohrrpgce] SVN: james/5080 Fix broken slice mutators. slices.bas compiles again now
James Paige
Bob at HamsterRepublic.com
Tue Feb 28 09:13:12 PST 2012
On Tue, Feb 28, 2012 at 09:04:21AM -0800, subversion at HamsterRepublic.com wrote:
> james
> 2012-02-28 09:04:21 -0800 (Tue, 28 Feb 2012)
> 57
> Fix broken slice mutators. slices.bas compiles again now
> ---
> U wip/slices.bas
I have several thoughts about these accessors and mutators.
the OHR code is riddled with crazy naming schemes and inconsistent
style, but slices.bas is wone of the few files that is well organized
and consistently style, so it would be nice to keep it that way.
What I notice especially is that all the existing slice related
functions all take the Slice ptr as the first argument. I would like it
if these new slice mutator functions would do the same.
There are other style issues like spaces instead of tabs, or names in
the form of SomethingSomethingSlice() instead of
sliceSomethingSomething() but those are of much less importance to me
than the argument order.
I was also thinking about null ptr checking. Most of the other slice
functions do explicit null ptr checking. In the past, this was a really
big deal for me because null ptr bugs used to be INSANELY frustrating to
debug. However, now that we can run in gdb, and use valgrind (and now
that I know how to do both of those things) I guess I don't have to be
the same stickler for null ptr checking that I used to be...
---
James Paige
More information about the Ohrrpgce
mailing list