~bzr-pqm/bzr/bzr.dev

2052.3.1 by John Arbash Meinel
Add tests to cleanup the copyright of all source files
1
# Copyright (C) 2005, 2006 Canonical Ltd
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
2
#
1 by mbp at sourcefrog
import from baz patch-364
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
7
#
1 by mbp at sourcefrog
import from baz patch-364
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
12
#
1 by mbp at sourcefrog
import from baz patch-364
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1 by mbp at sourcefrog
import from baz patch-364
16
1374 by Martin Pool
todo
17
# TODO: Could remember a bias towards whether a particular store is typically
18
# compressed or not.
19
711 by Martin Pool
- store docs
20
"""
1861.2.6 by Alexander Belchenko
branding: change Bazaar-NG to Bazaar
21
Stores are the main data-storage mechanism for Bazaar.
1 by mbp at sourcefrog
import from baz patch-364
22
23
A store is a simple write-once container indexed by a universally
711 by Martin Pool
- store docs
24
unique ID.
25
"""
1 by mbp at sourcefrog
import from baz patch-364
26
1442.1.51 by Robert Collins
teach iter about suffixes
27
import os
1185.1.41 by Robert Collins
massive patch from Alexander Belchenko - many PEP8 fixes, removes unused function uuid
28
1955.3.13 by John Arbash Meinel
Run the full test suite, and fix up any deprecation warnings.
29
from bzrlib import (
30
    errors,
3350.6.1 by Robert Collins
* New ``versionedfile.KeyMapper`` interface to abstract out the access to
31
    versionedfile,
1955.3.13 by John Arbash Meinel
Run the full test suite, and fix up any deprecation warnings.
32
    )
3224.5.26 by Andrew Bennetts
More minor import tidying suggested by pyflakes.
33
from bzrlib.errors import BzrError, UnlistableStore
1104 by Martin Pool
- Add a simple UIFactory
34
from bzrlib.trace import mutter
1 by mbp at sourcefrog
import from baz patch-364
35
36
######################################################################
37
# stores
38
39
class StoreError(Exception):
40
    pass
41
42
1185.11.1 by John Arbash Meinel
(broken) Transport work is merged in. Tests do not pass yet.
43
class Store(object):
907.1.1 by John Arbash Meinel
Reworking the Branch and Store code to support an abstracted filesystem layer.
44
    """This class represents the abstract storage layout for saving information.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
45
1 by mbp at sourcefrog
import from baz patch-364
46
    Files can be added, but not modified once they are in.  Typically
47
    the hash is used as the name, or something else known to be unique,
48
    such as a UUID.
907.1.1 by John Arbash Meinel
Reworking the Branch and Store code to support an abstracted filesystem layer.
49
    """
50
51
    def __len__(self):
52
        raise NotImplementedError('Children should define their length')
53
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
54
    def get(self, fileid, suffix=None):
1442.1.50 by Robert Collins
test get with suffixes
55
        """Returns a file reading from a particular entry.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
56
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
57
        If suffix is present, retrieve the named suffix for fileid.
1442.1.50 by Robert Collins
test get with suffixes
58
        """
59
        raise NotImplementedError
1442.1.35 by Robert Collins
convert all users of __getitem__ into TransportStores to use .get instead
60
907.1.1 by John Arbash Meinel
Reworking the Branch and Store code to support an abstracted filesystem layer.
61
    def __getitem__(self, fileid):
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
62
        """DEPRECATED. Please use .get(fileid) instead."""
907.1.1 by John Arbash Meinel
Reworking the Branch and Store code to support an abstracted filesystem layer.
63
        raise NotImplementedError
64
65
    def __iter__(self):
66
        raise NotImplementedError
67
907.1.43 by John Arbash Meinel
Restoring compatibility for Storage.add(file, fileid), it is a little arbitrary, and compatibility is better
68
    def add(self, f, fileid):
907.1.1 by John Arbash Meinel
Reworking the Branch and Store code to support an abstracted filesystem layer.
69
        """Add a file object f to the store accessible from the given fileid"""
1185.11.1 by John Arbash Meinel
(broken) Transport work is merged in. Tests do not pass yet.
70
        raise NotImplementedError('Children of Store must define their method of adding entries.')
907.1.1 by John Arbash Meinel
Reworking the Branch and Store code to support an abstracted filesystem layer.
71
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
72
    def has_id(self, fileid, suffix=None):
73
        """Return True or false for the presence of fileid in the store.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
74
75
        suffix, if present, is a per file suffix, i.e. for digital signature
1442.1.47 by Robert Collins
test for has with suffixed files
76
        data."""
1442.1.45 by Robert Collins
replace __contains__ calls in stores with has_id
77
        raise NotImplementedError
907.1.36 by John Arbash Meinel
Moving the multi-get functionality higher up into the Branch class.
78
1400.1.1 by Robert Collins
implement a basic test for the ui branch command from http servers
79
    def listable(self):
80
        """Return True if this store is able to be listed."""
1963.2.6 by Robey Pointer
pychecker is on crack; go back to using 'is None'.
81
        return (getattr(self, "__iter__", None) is not None)
1400.1.1 by Robert Collins
implement a basic test for the ui branch command from http servers
82
1563.2.14 by Robert Collins
Prepare weave store to delegate copy details to the versioned file.
83
    def copy_all_ids(self, store_from, pb=None):
84
        """Copy all the file ids from store_from into self."""
85
        if not store_from.listable():
86
            raise UnlistableStore(store_from)
87
        ids = []
88
        for count, file_id in enumerate(store_from):
89
            if pb:
90
                pb.update('listing files', count, count)
91
            ids.append(file_id)
92
        if pb:
93
            pb.clear()
94
        mutter('copy_all ids: %r', ids)
95
        self.copy_multi(store_from, ids, pb=pb)
96
1185.11.1 by John Arbash Meinel
(broken) Transport work is merged in. Tests do not pass yet.
97
    def copy_multi(self, other, ids, pb=None, permit_failure=False):
907.1.1 by John Arbash Meinel
Reworking the Branch and Store code to support an abstracted filesystem layer.
98
        """Copy texts for ids from other into self.
99
100
        If an id is present in self, it is skipped.  A count of copied
101
        ids is returned, which may be less than len(ids).
907.1.2 by John Arbash Meinel
Working on making Branch() do all of it's work over a Transport.
102
1185.11.1 by John Arbash Meinel
(broken) Transport work is merged in. Tests do not pass yet.
103
        :param other: Another Store object
907.1.2 by John Arbash Meinel
Working on making Branch() do all of it's work over a Transport.
104
        :param ids: A list of entry ids to be copied
4463.1.1 by Martin Pool
Update docstrings for recent progress changes
105
        :param pb: A ProgressTask object, if none is given, the default will be created.
1185.11.1 by John Arbash Meinel
(broken) Transport work is merged in. Tests do not pass yet.
106
        :param permit_failure: Allow missing entries to be ignored
107
        :return: (n_copied, [failed]) The number of entries copied successfully,
108
            followed by a list of entries which could not be copied (because they
109
            were missing)
907.1.1 by John Arbash Meinel
Reworking the Branch and Store code to support an abstracted filesystem layer.
110
        """
1185.79.2 by John Arbash Meinel
Adding progress bars to copy_all and copy_multi, fixing ordering of repository.clone() to pull inventories after weaves.
111
        if pb:
112
            pb.update('preparing to copy')
974.2.7 by aaron.bentley at utoronto
Merged from bzr.24
113
        failed = set()
1442.1.53 by Robert Collins
Unroll the multiple-copy logic enough to remove the duplicate iteration and yet retain the optimised gzip->gzip copy.
114
        count = 0
115
        for fileid in ids:
116
            count += 1
117
            if self.has_id(fileid):
118
                continue
119
            try:
1442.1.54 by Robert Collins
Teach store.copy_all about fileid suffixes
120
                self._copy_one(fileid, None, other, pb)
121
                for suffix in self._suffixes:
122
                    try:
123
                        self._copy_one(fileid, suffix, other, pb)
124
                    except KeyError:
125
                        pass
1185.79.2 by John Arbash Meinel
Adding progress bars to copy_all and copy_multi, fixing ordering of repository.clone() to pull inventories after weaves.
126
                if pb:
127
                    pb.update('copy', count, len(ids))
1442.1.53 by Robert Collins
Unroll the multiple-copy logic enough to remove the duplicate iteration and yet retain the optimised gzip->gzip copy.
128
            except KeyError:
129
                if permit_failure:
130
                    failed.add(fileid)
131
                else:
132
                    raise
1185.79.2 by John Arbash Meinel
Adding progress bars to copy_all and copy_multi, fixing ordering of repository.clone() to pull inventories after weaves.
133
        if pb:
134
            pb.clear()
1442.1.53 by Robert Collins
Unroll the multiple-copy logic enough to remove the duplicate iteration and yet retain the optimised gzip->gzip copy.
135
        return count, failed
136
1442.1.54 by Robert Collins
Teach store.copy_all about fileid suffixes
137
    def _copy_one(self, fileid, suffix, other, pb):
1442.1.53 by Robert Collins
Unroll the multiple-copy logic enough to remove the duplicate iteration and yet retain the optimised gzip->gzip copy.
138
        """Most generic copy-one object routine.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
139
1442.1.53 by Robert Collins
Unroll the multiple-copy logic enough to remove the duplicate iteration and yet retain the optimised gzip->gzip copy.
140
        Subclasses can override this to provide an optimised
141
        copy between their own instances. Such overriden routines
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
142
        should call this if they have no optimised facility for a
1442.1.53 by Robert Collins
Unroll the multiple-copy logic enough to remove the duplicate iteration and yet retain the optimised gzip->gzip copy.
143
        specific 'other'.
144
        """
1185.16.159 by John Arbash Meinel
Updated the stores, all tests pass, and a store doesn't have to be 100% compressed
145
        mutter('Store._copy_one: %r', fileid)
1442.1.54 by Robert Collins
Teach store.copy_all about fileid suffixes
146
        f = other.get(fileid, suffix)
147
        self.add(f, fileid, suffix)
1185.10.1 by Aaron Bentley
Added --basis option to bzr branch
148
1092.2.24 by Robert Collins
merge from martins newformat branch - brings in transport abstraction
149
150
class TransportStore(Store):
151
    """A TransportStore is a Store superclass for Stores that use Transports."""
152
1442.1.33 by Robert Collins
teach TransportStore.add to accept an optional file suffix, which does not alter the fileid.
153
    def add(self, f, fileid, suffix=None):
1442.1.28 by Robert Collins
pull up core TransportStore.add from TextStore.add and CompressedTextStore.add
154
        """Add contents of a file into the store.
155
1955.3.13 by John Arbash Meinel
Run the full test suite, and fix up any deprecation warnings.
156
        f -- A file-like object
1442.1.28 by Robert Collins
pull up core TransportStore.add from TextStore.add and CompressedTextStore.add
157
        """
1185.16.159 by John Arbash Meinel
Updated the stores, all tests pass, and a store doesn't have to be 100% compressed
158
        mutter("add store entry %r", fileid)
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
159
        names = self._id_to_names(fileid, suffix)
160
        if self._transport.has_any(names):
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
161
            raise BzrError("store %r already contains id %r"
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
162
                           % (self._transport.base, fileid))
1442.1.28 by Robert Collins
pull up core TransportStore.add from TextStore.add and CompressedTextStore.add
163
1185.16.159 by John Arbash Meinel
Updated the stores, all tests pass, and a store doesn't have to be 100% compressed
164
        # Most of the time, just adding the file will work
165
        # if we find a time where it fails, (because the dir
166
        # doesn't exist), then create the dir, and try again
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
167
        self._add(names[0], f)
168
169
    def _add(self, relpath, f):
170
        """Actually add the file to the given location.
171
        This should be overridden by children.
172
        """
173
        raise NotImplementedError('children need to implement this function.')
1442.1.28 by Robert Collins
pull up core TransportStore.add from TextStore.add and CompressedTextStore.add
174
1442.1.24 by Robert Collins
Pull up _check_id and _relpath from Text and CompressedText stores into TransportStore
175
    def _check_fileid(self, fileid):
3350.6.1 by Robert Collins
* New ``versionedfile.KeyMapper`` interface to abstract out the access to
176
        if type(fileid) != str:
177
            raise TypeError('Fileids should be bytestrings: %s %r' % (
178
                type(fileid), fileid))
1442.1.24 by Robert Collins
Pull up _check_id and _relpath from Text and CompressedText stores into TransportStore
179
        if '\\' in fileid or '/' in fileid:
180
            raise ValueError("invalid store id %r" % fileid)
181
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
182
    def _id_to_names(self, fileid, suffix):
183
        """Return the names in the expected order"""
1442.1.47 by Robert Collins
test for has with suffixed files
184
        if suffix is not None:
185
            fn = self._relpath(fileid, [suffix])
186
        else:
187
            fn = self._relpath(fileid)
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
188
1185.65.13 by Robert Collins
Merge from integration
189
        # FIXME RBC 20051128 this belongs in TextStore.
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
190
        fn_gz = fn + '.gz'
191
        if self._compressed:
192
            return fn_gz, fn
193
        else:
194
            return fn, fn_gz
195
196
    def has_id(self, fileid, suffix=None):
197
        """See Store.has_id."""
1651.1.5 by Martin Pool
Review cleanup of TransportStore.has_id
198
        return self._transport.has_any(self._id_to_names(fileid, suffix))
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
199
200
    def _get_name(self, fileid, suffix=None):
201
        """A special check, which returns the name of an existing file.
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
202
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
203
        This is similar in spirit to 'has_id', but it is designed
204
        to return information about which file the store has.
205
        """
206
        for name in self._id_to_names(fileid, suffix=suffix):
207
            if self._transport.has(name):
208
                return name
209
        return None
1442.1.38 by Robert Collins
unify __contains__ for TransportStore classes
210
1442.1.36 by Robert Collins
convert get() in TextStore and CompressedTextStore into a template method
211
    def _get(self, filename):
212
        """Return an vanilla file stream for clients to read from.
213
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
214
        This is the body of a template method on 'get', and should be
1442.1.36 by Robert Collins
convert get() in TextStore and CompressedTextStore into a template method
215
        implemented by subclasses.
216
        """
217
        raise NotImplementedError
218
1442.1.50 by Robert Collins
test get with suffixes
219
    def get(self, fileid, suffix=None):
220
        """See Store.get()."""
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
221
        names = self._id_to_names(fileid, suffix)
222
        for name in names:
223
            try:
224
                return self._get(name)
225
            except errors.NoSuchFile:
226
                pass
227
        raise KeyError(fileid)
1433 by Robert Collins
merge in and make incremental Gustavo Niemeyers nested log patch, and remove all bare exceptions in store and transport packages.
228
1185.58.4 by John Arbash Meinel
Added permission checking to Branch, and propogated that change into the stores.
229
    def __init__(self, a_transport, prefixed=False, compressed=False,
1185.80.1 by John Arbash Meinel
Text store and weave store both allow escaping fileid paths.
230
                 dir_mode=None, file_mode=None,
231
                 escaped=False):
1092.2.24 by Robert Collins
merge from martins newformat branch - brings in transport abstraction
232
        super(TransportStore, self).__init__()
1442.1.44 by Robert Collins
Many transport related tweaks:
233
        self._transport = a_transport
1442.1.25 by Robert Collins
Test TransportStore._relpath for simple cases: pull up _prefixed attribute as a result.
234
        self._prefixed = prefixed
1185.65.13 by Robert Collins
Merge from integration
235
        # FIXME RBC 20051128 this belongs in TextStore.
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
236
        self._compressed = compressed
1442.1.43 by Robert Collins
add registration of suffixes, in preparation for ensuring iteration is regular
237
        self._suffixes = set()
1185.80.1 by John Arbash Meinel
Text store and weave store both allow escaping fileid paths.
238
        self._escaped = escaped
1092.2.24 by Robert Collins
merge from martins newformat branch - brings in transport abstraction
239
1185.58.6 by John Arbash Meinel
Stores don't have to have a dir_mode or file_mode set
240
        # It is okay for these to be None, it just means they
241
        # will just use the filesystem defaults
242
        self._dir_mode = dir_mode
243
        self._file_mode = file_mode
3350.6.1 by Robert Collins
* New ``versionedfile.KeyMapper`` interface to abstract out the access to
244
        # Create a key mapper to use
245
        if escaped and prefixed:
246
            self._mapper = versionedfile.HashEscapedPrefixMapper()
247
        elif not escaped and prefixed:
248
            self._mapper = versionedfile.HashPrefixMapper()
249
        elif self._escaped:
3350.6.8 by Martin Pool
Change stray pdb calls to exceptions
250
            raise ValueError(
251
                "%r: escaped unprefixed stores are not permitted."
252
                % (self,))
1608.2.1 by Martin Pool
[merge] Storage filename escaping
253
        else:
3350.6.1 by Robert Collins
* New ``versionedfile.KeyMapper`` interface to abstract out the access to
254
            self._mapper = versionedfile.PrefixMapper()
1608.2.1 by Martin Pool
[merge] Storage filename escaping
255
1479 by Robert Collins
More quoting at the transport layer bugfixes.
256
    def _iter_files_recursive(self):
257
        """Iterate through the files in the transport."""
258
        for quoted_relpath in self._transport.iter_files_recursive():
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
259
            yield quoted_relpath
1479 by Robert Collins
More quoting at the transport layer bugfixes.
260
1442.1.51 by Robert Collins
teach iter about suffixes
261
    def __iter__(self):
1479 by Robert Collins
More quoting at the transport layer bugfixes.
262
        for relpath in self._iter_files_recursive():
1442.1.51 by Robert Collins
teach iter about suffixes
263
            # worst case is one of each suffix.
264
            name = os.path.basename(relpath)
265
            if name.endswith('.gz'):
266
                name = name[:-3]
267
            skip = False
268
            for count in range(len(self._suffixes)):
269
                for suffix in self._suffixes:
270
                    if name.endswith('.' + suffix):
271
                        skip = True
272
            if not skip:
3350.6.1 by Robert Collins
* New ``versionedfile.KeyMapper`` interface to abstract out the access to
273
                yield self._mapper.unmap(name)[0]
1442.1.51 by Robert Collins
teach iter about suffixes
274
1442.1.40 by Robert Collins
unify __len__() implementations for TransportStore classes
275
    def __len__(self):
1442.1.50 by Robert Collins
test get with suffixes
276
        return len(list(self.__iter__()))
1442.1.40 by Robert Collins
unify __len__() implementations for TransportStore classes
277
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
278
    def _relpath(self, fileid, suffixes=None):
1442.1.24 by Robert Collins
Pull up _check_id and _relpath from Text and CompressedText stores into TransportStore
279
        self._check_fileid(fileid)
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
280
        if suffixes:
281
            for suffix in suffixes:
282
                if not suffix in self._suffixes:
283
                    raise ValueError("Unregistered suffix %r" % suffix)
284
                self._check_fileid(suffix)
285
        else:
286
            suffixes = []
3350.6.1 by Robert Collins
* New ``versionedfile.KeyMapper`` interface to abstract out the access to
287
        path = self._mapper.map((fileid,))
3350.6.4 by Robert Collins
First cut at pluralised VersionedFiles. Some rather massive API incompatabilities, primarily because of the difficulty of coherence among competing stores.
288
        full_path = '.'.join([path] + suffixes)
289
        return full_path
1442.1.24 by Robert Collins
Pull up _check_id and _relpath from Text and CompressedText stores into TransportStore
290
1092.2.24 by Robert Collins
merge from martins newformat branch - brings in transport abstraction
291
    def __repr__(self):
292
        if self._transport is None:
293
            return "%s(None)" % (self.__class__.__name__)
294
        else:
295
            return "%s(%r)" % (self.__class__.__name__, self._transport.base)
296
297
    __str__ = __repr__
1185.10.1 by Aaron Bentley
Added --basis option to bzr branch
298
1400.1.1 by Robert Collins
implement a basic test for the ui branch command from http servers
299
    def listable(self):
300
        """Return True if this store is able to be listed."""
301
        return self._transport.listable()
302
1442.1.43 by Robert Collins
add registration of suffixes, in preparation for ensuring iteration is regular
303
    def register_suffix(self, suffix):
304
        """Register a suffix as being expected in this store."""
305
        self._check_fileid(suffix)
1185.16.157 by John Arbash Meinel
Added ability for TextStore to handle both compressed and uncompressed, it just looks for one type first
306
        if suffix == 'gz':
307
            raise ValueError('You cannot register the "gz" suffix.')
1442.1.43 by Robert Collins
add registration of suffixes, in preparation for ensuring iteration is regular
308
        self._suffixes.add(suffix)
309
1442.1.37 by Robert Collins
pull up total_size into TransportStore
310
    def total_size(self):
311
        """Return (count, bytes)
312
313
        This is the (compressed) size stored on disk, not the size of
314
        the content."""
315
        total = 0
316
        count = 0
1442.1.44 by Robert Collins
Many transport related tweaks:
317
        for relpath in self._transport.iter_files_recursive():
1442.1.37 by Robert Collins
pull up total_size into TransportStore
318
            count += 1
1442.1.44 by Robert Collins
Many transport related tweaks:
319
            total += self._transport.stat(relpath).st_size
3943.8.1 by Marius Kruger
remove all trailing whitespace from bzr source
320
1442.1.37 by Robert Collins
pull up total_size into TransportStore
321
        return count, total