Re: refactory and file names

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> Hi,
> 
> On Mon, 2015-11-23 at 12:44 -0500, Frediano Ziglio wrote:
> > Hi,
> >   as part of the refactory there are some file renames. Actually
> > the patches with files renames where removed if the file was already
> > present and all new files has already the new names.
> > 
> > The files should be (as the original changes) renamed like
> > 
> > red_parse_qxl.c -> red-parse-qxl.c
> > red_record_qxl.c -> red-record-qxl.c
> > red_replay_qxl.c -> red-replay-qxl.c
> > red_parse_qxl.h -> red-parse-qxl.h
> > red_record_qxl.h -> red-record-qxl.h
> > red_replay_qxl.h -> red-replay-qxl.h
> > reds_stream.c -> reds-stream.c
> > reds_stream.h -> reds-stream.h
> > red_worker.h -> red-worker.h
> > red_worker.c -> red-worker.c
> > lz4_encoder.c -> lz4-encoder.c
> > lz4_encoder.h -> lz4-encoder.h
> > red_dispatcher.c -> red-dispatcher.c
> > red_dispatcher.h -> red-dispatcher.h
> > red_channel.c -> red-channel.c
> > red_channel.h -> red-channel.h
> > red_common.h -> common.h
> > red_client_cache.h -> cache_item.tmpl.c
> > 
> > Also there are these patches
> > http://cgit.freedesktop.org/~fziglio/spice-server/commit/?h=refactory&id=d5ddf
> > 4c9d36c9b9238f86079f8668d38d911f54c
> > http://cgit.freedesktop.org/~fziglio/spice-server/commit/?h=refactory&id=ac2f5
> > 6d3faf64f0cb497c3245d2d997f235cdc87
> > 
> > Now... what's the rationale of these patches?
> > 
> > Prefer hyphen ('-') to underscore ('_') is surely one.
> I am ok with both, but spice-gtk uses '-' so maybe for the consistency... I
> am
> not sure if it is a strong enough argument.
> 
> > 
> > The other, which I really don't like is that all template sources should
> > finish with .template. I don't like as this force users to set their
> > environment in order to have syntax highlight of C files.
> > 
> I would prefer to keep the ".tmpl.c" suffix.
> 
> Pavel

Actually mostly are *_tmpl.c but I vote for *.tmpl.c too.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]