hmmm...

> which matches the expected results. So Xiphos is getting around calling 
> setIntros(1) by instead disabling auto-normalizing. It's odd to me that you 
> can set a key to a value in the intros while setIntros is false. That doesn't 
> seem like normalizing to me, that seems more like bounds checking, but that's 
> not necessarily a bug in the API, possibly it's a bug in my understanding of 
> intended behavior.

What should the behaviour be when intros are off and you try key->setVerse(0)? 
should it then go to the last verse in the previous chapter or bump up to verse 
1 in the current chapter? If we were already at verse 1 & we decremented the 
key, it should go to the previous chapter's last verse, but in this case (given 
it's been set explicitly to verse 0), would it be more correct to bump up to 
the "first" valid verse in the current chapter (so, verse 0/intro if intros are 
switched on, or verse 1 if intros are switched off)?

My thought is that it shouldn't allow it to stay at verse 0, but when 
normalize() is called, it should "fix" it to a valid verse for that 
versification system (given intros are switched off and there is no such thing 
as verse 0 in that case)?

But then again, I am probably in the same boat as Greg and maybe I just don't 
understand the intended behaviour of the API in this situation?


Thanks, ybic
        nic...  :)
_______________________________________________
sword-devel mailing list: sword-devel@crosswire.org
http://www.crosswire.org/mailman/listinfo/sword-devel
Instructions to unsubscribe/change your settings at above page

Reply via email to