src/sitemapservice/sitemapelement.cc
std::string UrlElement::ConvertFrequencyToString(const UrlElement::ChangeFreq& changefreq) {
  switch (changefreq) {
    case UrlElement::ALWAYS:   return "always";
    case UrlElement::HOURLY:   return "hourly";
    case UrlElement::DAILY:    return "daily";
    case UrlElement::WEEKLY:   return "weekly";
    case UrlElement::MONTHLY:  return "monthly";
    case UrlElement::YEARLY:   return "yearly";
    case UrlElement::NEVER:    return "never";
    default:            return ""; // log error here!
  }
}
// I think rename it to ConvertFrequencyToString is better!
I think rename it to ConvertFrequencyToString is better!
I think rename it to ConvertFrequencyToString is better!
std::string UrlElement::ConvertFrequencyToString(const UrlElement::ChangeFreq& changefreq) {
  switch (changefreq) {
    case UrlElement::ALWAYS:   return "always";
    case UrlElement::HOURLY:   return "hourly";
    case UrlElement::DAILY:    return "daily";
    case UrlElement::WEEKLY:   return "weekly";
    case UrlElement::MONTHLY:  return "monthly";
    case UrlElement::YEARLY:   return "yearly";
    case UrlElement::NEVER:    return "never";
    default:            return ""; 
  }
}
std :: string UrlElement :: ConvertFrequencyToString ( const UrlElement :: ChangeFreq & changefreq ) { switch ( changefreq ) { case UrlElement :: ALWAYS : return " " ; case UrlElement :: HOURLY : return " " ; case UrlElement :: DAILY : return " " ; case UrlElement :: WEEKLY : return " " ; case UrlElement :: MONTHLY : return " " ; case UrlElement :: YEARLY : return " " ; case UrlElement :: NEVER : return " " ; default : return " " ; } }
I think rename it to ConvertFrequencyToString is better!
I think rename it to ConvertFrequencyToString is better!
// log error here!