brewtarget-devs team mailing list archive
Mailing list archive
Re: displayOG and displayFG
I think the reason they are different is that displayFG used to display
apparent FG as read by a refractometer instead of the true FG. This is why
it includes the OG as a parameter. It seems like the OG is not considered
On Mon, Sep 1, 2014 at 2:29 PM, mik firestone <mikfire@xxxxxxxxx> wrote:
> Based on the comments, I am guessing these two methods were once
> significantly different.
> My reading is that now the bodies of the two methods are almost identical.
> displayFG has an extra check to make sure the supplied OG is reasonable.
> I am currently in the middle of a huge refactoring. The refactoring should
> make things a lot easier to add later. It would make things easier for
> this effort if I could either collapse the two methods into one, or if I
> could simply fold it all into displayAmount().
> Is there any significant difference between these two methods, and is
> there any reason I shouldn't merge them?
> In a world of ninja v. pirate, I pilot a Gundam
> Mailing list: https://launchpad.net/~brewtarget-devs
> Post to : brewtarget-devs@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~brewtarget-devs
> More help : https://help.launchpad.net/ListHelp
Philip G. Lee