Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [RFC] Code Style Questions
@ 2025-06-13  1:59 softworkz .
  0 siblings, 0 replies; only message in thread
From: softworkz . @ 2025-06-13  1:59 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

Hi,

I have some questions regarding code style.


1. White space
==============

Do we prefer this

  (float)(*times)[i] / 1000000, (float)(*times)[I - 1] / 1000000);

or this

  (float)(*times)[i]/1000000, (float)(*times)[i-1]/1000000);



2. Multiline Ternary Expressions
================================

Do we prefer this:

end_pts = (INT64_MAX - end_pts >= seg->reference_stream_first_pts)
            ? seg->reference_stream_first_pts
            : INT64_MAX - end_pts;

or this

end_pts = (INT64_MAX - end_pts >= seg->reference_stream_first_pts) ?
            seg->reference_stream_first_pts :
            INT64_MAX - end_pts;


Thanks
sw

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2025-06-13  1:59 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-06-13  1:59 [FFmpeg-devel] [RFC] Code Style Questions softworkz .

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git