Bug 217601

Summary: kunitconversion - millimeters of mecrury (mmHg) missing
Product: [Frameworks and Libraries] kdelibs Reporter: Andrey Borzenkov <arvidjaar>
Component: generalAssignee: Petri Damstén <petri.damsten>
Status: RESOLVED FIXED    
Severity: wishlist CC: annma, jlayt, mpyne, petri.damsten
Priority: NOR    
Version: 4.3   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Patch to add mmHg units to kunitconversion

Description Andrey Borzenkov 2009-12-06 16:23:42 UTC 
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

Standard unit for atmospheric pressure in Russia is millimetres of mercury (mmHg). This is missing as of current 4.4 beta. Are there reasons to not add it? Will patch be accepted?
Comment 1 Michael Pyne 2009-12-06 21:35:37 UTC 
I've heard of the unit, I'm sure it's merely an oversight. If you have a patch let's see it. :)
Comment 2 Anne-Marie Mahfouf 2009-12-07 08:33:16 UTC 
Yes it would be good to have the patch so this is not forgotten for 4.5 (not allowed to add new strings in 4.4)
Comment 3 Andrey Borzenkov 2009-12-11 16:19:23 UTC 
Created attachment 38992 [details]
Patch to add mmHg units to kunitconversion

Patch attached. Unfortunately I do not have infrastructure to actually build it.
Comment 4 John Layt 2009-12-11 17:45:24 UTC 
Need to assign to petri.damsten@gmail.com as author/maintainer of code, but he doesn't seem to be registered as yet.  I'll cc: him.

I'll also get Petri to raise a request in bugzilla against bugs.kde.org to request adding a KUnitConverter category to kdelibs Product, with default assignment to Petri.
Comment 5 Petri Damstén 2010-02-17 09:01:40 UTC 
SVN commit 1091623 by pdamsten:

Add millimeters of mecrury (mmHg)
patch from Andrey Borzenkov
BUG: 217601

 M  +1 -1      converter.h  
 M  +12 -3     pressure.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1091623