[Spice-devel] refactory and file names

[Pavel Grunt]

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
> What's the user preference?
> 
> Frediano
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel