Re: Review Request 430: [1/1] Locale: MOF

[Stephen Gallagher]

>  On May 19, 2014, 2:06 p.m., Jan Safranek wrote:
>  > The MOF file is now much better, thanks. On the other hand, I think you forgot to
> update the .c files appropriately :). 
That's intentional - I want to be sure the model is 'correct' and
'final' before I start modifying .c files... Stay tuned:)


>  On May 19, 2014, 2:06 p.m., Jan Safranek wrote:
>  > mof/60_LMI_Locale.mof, line 165
>  >
> <http://reviewboard-openlmi.rhcloud.com/r/430/diff/1-2/?file=2559#file2559...
>  >
>  >     The name suggests there can be more than one option. How are they separated?
Good point. They're comma separated. I'll update the description accordingly.
Actually, it's also possible to set more than one layout in the same way, so it makes
sense also renaming 'string X11Layout;' to 'string X11Layouts;' and
document that fact too.


- Vitezslav


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/430/#review2652
-----------------------------------------------------------


On May 19, 2014, 11:23 a.m., Vitezslav Crhonek wrote:
>  
>  -----------------------------------------------------------
>  This is an automatically generated e-mail. To reply, visit:
>  http://reviewboard-openlmi.rhcloud.com/r/430/
>  -----------------------------------------------------------
>  
>  (Updated May 19, 2014, 11:23 a.m.)
>  
>  
>  Review request for OpenLMI Developers.
>  
>  
>  Repository: openlmi-providers
>  
>  
>  Description
>  -------
>  
>  Locale: MOF
>  
>  
>  Diffs
>  -----
>  
>    mof/60_LMI_Locale.mof PRE-CREATION 
>  
>  Diff: http://reviewboard-openlmi.rhcloud.com/r/430/diff/
>  
>  
>  Testing
>  -------
>  
>  
>  Thanks,
>  
>  Vitezslav Crhonek
>  
>