[petsc-dev] naming of DM specific constructors is backwards

Dmitry Karpeev karpeev at mcs.anl.gov
Thu Sep 13 21:25:09 CDT 2012


On Thu, Sep 13, 2012 at 9:12 PM, Jed Brown <jedbrown at mcs.anl.gov> wrote:

> Constructor tab completion is better for DMCreateDA().
>
But will clash with all the other methods of the form DMDAGetXXX() etc.
I would vote for converting MatCreateSeqAIJ to MatSeqAIJCreate().
Note also that Mat*Create will Tab-expand to all possible Mat constructors
in an emacs tag search.

Dmitry.



>  On Sep 13, 2012 9:05 PM, "Barry Smith" <bsmith at mcs.anl.gov> wrote:
>
>>
>>    MatCreateSeqAIJ()  etc etc etc
>>
>>     but
>>
>>     DMDACreate(), DMRedundantCreate() etc
>>
>>
>>     Now given that normally we list the subclass name right after the
>> base class in methods, for example KSPGMRESSetRestart() not
>> KSPSetRestartGMRES() it might make sense to always follow the DMXXXCreate()
>> style and change ALL the other sub constructor names to the DM style. Boy I
>> know Wolfgang will not be happy with that :-)
>>
>>     Should we just fix the DM sub constructors to follow the style of all
>> the other ones?
>>
>>     Barry
>>
>>
>>
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/petsc-dev/attachments/20120913/436f1f89/attachment.html>


More information about the petsc-dev mailing list