[PATCH] Print nice names for nitrox mixes in the divelist

Joakim Bygdell j.bygdell at gmail.com
Sat Aug 16 08:36:36 PDT 2014


On 16 Aug 2014, at 17:33, Dirk Hohndel <dirk at hohndel.org> wrote:

> On Sat, Aug 16, 2014 at 09:30:58AM -0600, Linus Torvalds wrote:
>> On Sat, Aug 16, 2014 at 7:50 AM, Dirk Hohndel <dirk at hohndel.org> wrote:
>>> 
>>> This does cost us more space (we always struggle with not wasting screen
>>> space), but I think the value outweighs the issue.
>> 
>> I disagree. I disagree *violently* when it comes to the idiotic double EAN.
>> 
>> The string "EAN28...EAN50" is a *hell* of a lot bigger than "28...50",
>> and won't possibly fit in any sanely sized divelist column.
>> 
>> So I think this is a horrible change.
> 
> How about EAN28...50 ?
> 
> /D

Or we deal with this the same way we deal with trimix, only show one, which for nitrox would be the weakest one.
That way we always only show one gas regardless of if it trimix, nitrox or air.

/Jocke



More information about the subsurface mailing list