uci-cbcl/genomix — pull request #53 by jakebiesinger (2013-11-05 08:41:35+00:00)
Title: Fix PathMerge problem

@anbangx @JavierJia @Nan-Zhang @Elmira88

This PR is to fix the regression introduced by #51 commit 447146c. The stupid commit I've added here isn't the right way to fix the bug but it works for now. I'm adding this PR so people can use this commit to directly test things.

When you do your upcoming testing, everyone, base it on this branch, please.

@anbangx I feel like it would be easier for us to keep track of these values if instead of a `byte` member variable, we just had the missing values initialized as `null`. That involves more allocation and therefore GC and might make some pieces harder to deal with (you can't directly get contained values in e.g., `outgoingMsg`) but it would sure make reading/writing easier/ You'd just have to check if the value is `null` rather than actually read anything.

I've noticed in recent builds that I get a malformed byte error from HashMapWritable-- I bet there is a similar (or same?) issue there.