Skip to content

  • Projeler
  • Gruplar
  • Parçacıklar
  • Yardım
    • Yükleniyor...
  • Oturum aç / Kaydol
C
core
  • Proje
    • Proje
    • Ayrıntılar
    • Etkinlik
    • Cycle Analytics
  • Depo (repository)
    • Depo (repository)
    • Dosyalar
    • Kayıtlar (commit)
    • Dallar (branch)
    • Etiketler
    • Katkıda bulunanlar
    • Grafik
    • Karşılaştır
    • Grafikler
  • Konular (issue) 0
    • Konular (issue) 0
    • Liste
    • Pano
    • Etiketler
    • Kilometre Taşları
  • Birleştirme (merge) Talepleri 0
    • Birleştirme (merge) Talepleri 0
  • CI / CD
    • CI / CD
    • İş akışları (pipeline)
    • İşler
    • Zamanlamalar
    • Grafikler
  • Paketler
    • Paketler
  • Wiki
    • Wiki
  • Parçacıklar
    • Parçacıklar
  • Üyeler
    • Üyeler
  • Collapse sidebar
  • Etkinlik
  • Grafik
  • Grafikler
  • Yeni bir konu (issue) oluştur
  • İşler
  • Kayıtlar (commit)
  • Konu (issue) Panoları
  • LibreOffice
  • core
  • Repository

Dal (branch)/etiket değiştir
  • core
  • sc
  • inc
  • dpcache.hxx
Dosya bul
BlameHistoryPermalink
  • Dennis Francis's avatar
    tdf#124676 : use case-insensitive normalization of... · 9da59611
    Dennis Francis Nis 09, 2019 yazdı
    ScDPCache field labels, else on export to xlsx,
    Excel will fail to load the pivot table due to case-insensitive
    duplicate field labels in the pivotCacheDefinition1.xml.
    
    This could be done just for xlsx export filter, but we do normalization
    in dpcache.cxx anyway and it would not hurt if we do a case-insensitive
    normalization here.
    
    The private member ScDPCache::AddLabel had code duplication and
    more importantly it is called in loop for every label in the database
    so results in O(n^2) time complexity where n is the number of labels,
    so removed it to reuse normalizeLabels() at the only call-site.
    
    Also added a unit test that checks case-insensitive normalization.
    
    Change-Id: Id563dee232a98a2aea9f4fc29254f6942e1c5ba7
    Reviewed-on: https://gerrit.libreoffice.org/70498Reviewed-by: 's avatarMike Kaganski <mike.kaganski@collabora.com>
    Tested-by: Jenkins
    Reviewed-by: 's avatarDennis Francis <dennis.francis@collabora.com>
    (cherry picked from commit 238cadd3)
    Reviewed-on: https://gerrit.libreoffice.org/70703
    9da59611
dpcache.hxx 6.89 KB
DüzenleWeb IDE

Replace dpcache.hxx

Sürükleyip bırakarak bir dosya ekle veya yüklemek için tıklayın


İptal
Çatalınızda (fork) yeni bir dal (branch) oluşturulacak ve yeni bir birleştirme (merge) talebi başlatılacak.