From: Junio C Hamano Date: Wed, 08 Apr 2020 00:11:50 GMT Subject: Re: [PATCH v5 05/22] sequencer: make read_oneliner() accept flags Message-ID: In-Reply-To: Denton Liu writes: > In a future commit, we will need read_oneliner() to accept flags other > than just `skip_if_empty`. Instead of having an argument for each flag, > teach read_oneliner() to accept the bitfield `flags` instead. For now, > only recognize the `READ_ONELINER_SKIP_IF_EMPTY` flag. More flags will > be added in a future commit. Good move. Looked sensible. Thanks.